You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 4, 2019. It is now read-only.
To avoid conflicting with default tor configuration and with the Tor Browser, we should run the tor subprocess (of #473) with a custom socks port number that is not likely to conflict. The tor daemon uses 9050 by default, and the Tor Browser uses 9150 by default, so maybe we should use 9250. Ideally, we could use a different number for release/beta/dev builds so that developers can run (say) brave-beta with Tor support at the same time as developing it.
This is a short-term stop-gap until we have a nice way to set the socks proxy address only after the tor daemon has launched (which we will want for brave/browser-laptop#12936 unless we implement #469 first), and to cause Tor tabs to give meaningful feedback to users when tor is still launching rather than either failing cryptically or falling back to no proxy.
The text was updated successfully, but these errors were encountered:
To avoid conflicting with default tor configuration and with the Tor Browser, we should run the tor subprocess (of #473) with a custom socks port number that is not likely to conflict. The tor daemon uses 9050 by default, and the Tor Browser uses 9150 by default, so maybe we should use 9250. Ideally, we could use a different number for release/beta/dev builds so that developers can run (say) brave-beta with Tor support at the same time as developing it.
This is a short-term stop-gap until we have a nice way to set the socks proxy address only after the tor daemon has launched (which we will want for brave/browser-laptop#12936 unless we implement #469 first), and to cause Tor tabs to give meaningful feedback to users when tor is still launching rather than either failing cryptically or falling back to no proxy.
The text was updated successfully, but these errors were encountered: