Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

When user uses custom electrum server, make him aware that his addresses/txids sre still leaked to a Push Notificatinos serer

See original GitHub issue

@ncoelho i think we need your brilliance in making this work

or, quote:

I highly suggest that if someone sets a custom Electrum server, the push functionality is turned off and has to explicitly be opt-in

Issue Analytics

State:
Created a year ago
Reactions:5
Comments:9 (4 by maintainers)

Top GitHub Comments

1reaction

ncoelhocommented, Jul 17, 2022

If pn are off, nothing is sent to ground control.

On Sun, 17 Jul 2022 at 16:43, satrinity402 @.***> wrote:

Any updates here @Overtorment https://github.com/Overtorment? Were you able to find out if disabling push notifications still leaked Txid’s?

— Reply to this email directly, view it on GitHub https://github.com/BlueWallet/BlueWallet/issues/4806#issuecomment-1186538082, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJOLHLRVYGPVM7S3WWZQKLVUQLYJANCNFSM5ZITXPGQ . You are receiving this because you were mentioned.Message ID: @.***>

0reactions

Overtormentcommented, Jul 21, 2022

also, nothing is ever sent to PN server until the user presses OK on this dialog box: (its displayed when user goes to Receive screen)

perhaps this message should be reworded to explicitly include a statement that PushNotifications mean that there is a PN server that needs to see your addresses to track them to actually send notifications