Skip to content

Conversation

@daimon99
Copy link

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #20 into master will decrease coverage by 2.95%.
The diff coverage is 46.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
- Coverage   84.03%   81.08%   -2.96%     
==========================================
  Files           2        2              
  Lines         213      222       +9     
  Branches       27       31       +4     
==========================================
+ Hits          179      180       +1     
- Misses         26       30       +4     
- Partials        8       12       +4
Impacted Files Coverage Δ
src/remote_pdb.py 68.42% <46.66%> (-6%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57ed66b...04d4336. Read the comment docs.

@ionelmc
Copy link
Owner

ionelmc commented Jan 12, 2020

If there's support for uds I would like to also be supported in the set_trace API.

Also, it could be less verbose, eg: the host argument could also work as a path.

# ambiguous:
set_trace('foobar')  # will work as hostname

# unambiguous:
set_trace('/foo/bar')  # can only be a path since it has slashes
set_trace('foo/bar')  # can only be a path since it has slashes
set_trace('foobar', uds=True)  # special flag to switch to uds
# perhaps "unix" or "unix_socket" instead of "uds" to be more clear?

@holtgrewe
Copy link

Hi, what's the status of this? I'd like to use it in a HPC cluster setting and here using TCP ports is a securit issue.

@heroesm
Copy link

heroesm commented Sep 21, 2020

#27

@ionelmc ionelmc mentioned this pull request Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants