Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved error messages, now printing failed to resolve hostnames and IP addresses. #32

Closed
wants to merge 6 commits into from

Conversation

yurivict
Copy link

No description provided.

yurivict and others added 6 commits March 24, 2015 15:32
1. fork(2) and getpid(2) are called (by libc?) through syscall(2)
   and torsocks failed them since syscall hook is missing them.
2. The right for dlopen version of libc.so wasn't found by configure script
@dgoulet
Copy link
Owner

dgoulet commented Jun 13, 2016

3 commits in there are removing stuff that were added by the first commit...?

@yurivict
Copy link
Author

Sorry, some commits from another PR got accidentally mixed in here.

Are you able to integrate only these two commits: 21e2567, c0ab64b ?

islahbrasi referenced this pull request Jun 16, 2016
This patch adds two new arguments for the torsocks script: --address
(-a), for specifying the address of the Tor instance to use, and --port
(-P), for specifying the port of the Tor instance to use.
@dgoulet
Copy link
Owner

dgoulet commented Feb 24, 2017

I'm not too fan of this because by design torsocks shouldn't print addr/port in the logs for safety concerns like tor does. I would take a patch for adding "SafeLogging" like tor has and then we can do such a thing.

@dgoulet dgoulet closed this Feb 24, 2017
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.

2 participants