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

SOCKS4 failure message #32

Closed
keroserene opened this issue Mar 25, 2016 · 3 comments
Closed

SOCKS4 failure message #32

keroserene opened this issue Mar 25, 2016 · 3 comments
Labels

Comments

Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@keroserene
Copy link
Owner

@keroserene keroserene commented Mar 25, 2016

Sometimes, Tor emits this:

[warn] The connection to the SOCKS4 proxy server at 127.0.0.1:$someport just failed. Make sure that the proxy server is up and running.

At which point no more handlers fire. Looking at the Tor source, this happens when SOCKS is "marked for close". This is inconvenient because then the client must be restarted for connectivity to work again.

@keroserene keroserene added the bug label Mar 25, 2016
@arlolra
Copy link
Collaborator

@arlolra arlolra commented Jun 15, 2016

Are you still experiencing this? Is it reproducible?

@keroserene
Copy link
Owner Author

@keroserene keroserene commented Jun 16, 2016

I experienced this sporadically while working on multiplexing. Right now it seems fine though.

It seems to be 100% reproducible if I explicitly readPipe.Close() on the webrtc peer. My guess is that when the copyLoop sends an error or EOF at SOCKS, Tor interprets that as an unexpected situation, "marks for close / deletion" and so the SOCKS4 server disappears.

Maybe there's a way to account for this in main() to also be able to recover the initial SOCKS listening? (this is not in the goptlib example, which we were originally based on) When the SOCKS connection is closed normally in handler, there's doesn't seem to be an issue.

But, should test more possibilities / longer times.

@arlolra
Copy link
Collaborator

@arlolra arlolra commented Mar 22, 2018

@arlolra arlolra closed this Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment