Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

On certain networks Brave fails to connect to tor #14560

Closed
riastradh-brave opened this issue Jun 26, 2018 · 3 comments
Closed

On certain networks Brave fails to connect to tor #14560

riastradh-brave opened this issue Jun 26, 2018 · 3 comments

Comments

@riastradh-brave
Copy link
Contributor

Description

The outward-facing symptom is that Brave fails to connect to Tor.

The logs reveal that the tor daemon connects to the Tor network, but then Brave fails to connect to the tor daemon's control channel:

tor: control socket error: Error: getaddrinfo ENOTFOUND localhost undefined:55219
tor: control socket error: Error: getaddrinfo ENOTFOUND localhost undefined:55219
tor: control socket closed early

Steps to Reproduce

  1. Break into David's home network.
  2. Launch Brave.
  3. Open a private tab with Tor and try to use it.

Actual result:
Brave pops up a 'failed to connect to Tor' dialogue box and doesn't work.

Expected result:
Brave successfully uses the Tor network.

Reproduces how often:
Only on David's home network.

Brave Version

about:brave info:
some recent 0.23.x build with Tor support

Reproducible on current live release:
yes

Additional Information

Nobody has seen this outside David's home network. I suspect there is a broken DNS configuration that is causing localhost -- which nodejs socket.connect({port: 12345}) uses by default -- to resolve to something other than 127.0.0.1, or to fail to resolve.

@riastradh-brave
Copy link
Contributor Author

Possible fix: #14554 Will close this issue if it works.

@riastradh-brave
Copy link
Contributor Author

(To be clear to any literal-minded readers: I do not actually suggest breaking into David's home network. That would violate a few ethical principles of QA, and it might even violate some broader ethical frameworks too.)

@bsclifton bsclifton added this to the 0.23.x (Beta Channel) milestone Jun 26, 2018
@bsclifton
Copy link
Member

bsclifton commented Jun 27, 2018

Fix verified by @davidtemkin! 😄 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants