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

Data leakage when funding Bisq #1559

Closed
ronohara opened this issue May 27, 2018 · 3 comments
Closed

Data leakage when funding Bisq #1559

ronohara opened this issue May 27, 2018 · 3 comments

Comments

@ronohara
Copy link

ronohara commented May 27, 2018

When you scan the QR code to fund your Bisq wallet, using the defaults, it sets a description of "Fund Bisq wallet"... or at least it does when you send from Mycellium.

Presumably this is done using the payment protocol, but it leaks information. That payment protocol transfer is clear text. Easily intercepted on the network. Correlation of metadata then lets an attacker know that you are using Bisq, and how much your wallet is funded by. If it is a big amount, you become a target.

I suggest that using the payment protocol becomes optional. Just a QR code of the receiving address should be the default setting.

@ManfredKarrer
Copy link
Member

No that is not done with the payment protocol but the Bitcoin URI can contain a message, so it is just part of the QR code and only shared between Bisq and the scanning wallet.
Beside that you can copy / paste the address as well.

@ronohara
Copy link
Author

Ok - and using a QR code, that is not via the IP network. Good...

BUT - I would rather not have any description stored in my phone. Phones are insecure and subject to seizure/theft. Copy/paste does not work from a desktop (Bisq) to a phone. Only on the same device,

I have found a weird different solution.. Install Electrum and restore the Mycellium seed. Then you have a wallet that exists as a dual view. One on the phone, the other on the desktop - but just one wallet.
Still playing with that approach to see if there are any traps... but cut/paste to Electrum would work

@ManfredKarrer
Copy link
Member

You should be able on the phone to edit/delete the label text.

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

2 participants