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

electrum leaks when socks proxy is used #224

Closed
asn-d6 opened this issue May 21, 2013 · 7 comments
Closed

electrum leaks when socks proxy is used #224

asn-d6 opened this issue May 21, 2013 · 7 comments
Assignees

Comments

@asn-d6
Copy link

asn-d6 commented May 21, 2013

Hi there,

if you configure electrum with a SOCKS5 proxy, you get both DNS and HTTP leaks. Electrum does DNS requests for services like electrum.org, and does direct HTTP queries to some bitcoin exchanges to learn the current rates.

This is a quite serious problem, since Electrum is advertised as "secure", and a leaking proxy is a pretty big issue for a bitcoin client IMO.

@ghost
Copy link

ghost commented May 22, 2013

Not only does it leak but for me on Winx64 it actually connected to whatever preemptive address was set and did NOT go through the proxy. This is very serious.

@asn-d6
Copy link
Author

asn-d6 commented May 23, 2013

Indeed. FWIW, this is actually old news to the developers:
#207
#147

"Safe" and "Tested and audited" just like the website says.

@ghost ghost assigned ecdsa Jun 17, 2013
@ser
Copy link
Contributor

ser commented Aug 16, 2013

I have noticed proxy is being respected only when you use unencrypted TCP.

SSL, HTTP, HTTPS connections are going directly even if proxy is checked, which is really serious security issue!

@armisamimi
Copy link

I can confirm what @ser mentioned. This makes proxy support useless, since using SSL without tor is better than using tor without SSL.
Another issue is that if tor is not running, electrum still connects to its servers (which means there is no way to verify that information is actually going through tor).

@ecdsa
Copy link
Member

ecdsa commented Sep 27, 2013

the http requests to get the exchange rate are now part of a plugin, and no longer performed by default.
I'm still working on this, though

@ecdsa
Copy link
Member

ecdsa commented Oct 1, 2013

one more fixed here: 89a9ce2
the dns issue remains, though

@ecdsa
Copy link
Member

ecdsa commented Oct 1, 2013

http leaks are fixed.
all that remains here is a dupe of #147, so I'm closing this issue

@ecdsa ecdsa closed this as completed Oct 1, 2013
UdjinM6 pushed a commit to UdjinM6/electrum-dash that referenced this issue Oct 1, 2023
dash_ps: allow set absolute denoms count to mix
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

No branches or pull requests

4 participants