Bug fix in WalletSetup

so far i understand your version is working bc another bug in the
walletsetup where the Socks5MultiDiscovery is not used. so u use default
dnsdiscover which leaks dns. that works only with SafeSocks 0.
if using Socks5MultiDiscovery as is it should i only get 4 nodes form dns seeds.

Signed-off-by: Mike Rosseel <mike@eon-consult.be>
This commit is contained in:
Mike Rosseel 2017-11-04 16:33:20 +01:00
parent 9ff586c973
commit f1afee48dc
No known key found for this signature in database
GPG key ID: 1D9920AF2A8E6BF0

View file

@ -322,7 +322,7 @@ public class WalletsSetup {
// be private by default when using proxy/tor. However, the seedpeers
// could become outdated, so it is important that the user be able to
// disable it, but should be made aware of the reduced privacy.
if (socks5Proxy != null && !usePeerNodes && regTestHost != RegTestHost.LOCALHOST) {
if (socks5Proxy != null && !usePeerNodes || regTestHost == RegTestHost.REG_TEST_SERVER) {
// SeedPeers uses hard coded stable addresses (from MainNetParams). It should be updated from time to time.
walletConfig.setDiscovery(new Socks5MultiDiscovery(socks5Proxy, params, socks5DiscoverMode));
}