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

Auto deshield transaction #369

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

Duddino
Copy link
Member

@Duddino Duddino commented May 20, 2024

Abstract

This PR introduces auto deshielding.
Some places like exchanges don't support shield transactions. To circumvent this, MPW is set to automatically deshield to a newly generated address before sending.
Closes #337

Can this transaction be tracked to you?

The burner address is generated using a new change address, which MPW will not reuse. Tools like https://chainz.cryptoid.info/pivx/#!wallets will not work.
However these addresses can be easily tracked if the xpub is known. As of right now the xpub is advertised to explorers, while we only allow a set of trusted explorers, we can't be 100% sure they're not tracking user addresses, although very unlikely. If absolute privacy is needed, as always, it's best to self host an explorer.

We could technically make an address unlinked to the xpub or using a special hardened path, however this makes things more difficult to recover should the first deshield transaction go through and the second one fail.
If this were to happen right now, the deshielded funds would simply be available as regular balance.

Other changes

Shield addresses will now show as red in transfermenu if shield is not enabled

TODO

  • Add unit tests
  • Figure out what happens when sending with auto shield enabled to a shield address

Testing

  • Send to a transparent address using auto shield. Two transactions should be created and the shield balance should be used
  • Send to an exchange address/xpub
  • Send to a shield address and test that it throws a user friendly error

Copy link

netlify bot commented May 20, 2024

Deploy Preview for cheery-moxie-4f1121 ready!

Name Link
🔨 Latest commit eb7cb6a
🔍 Latest deploy log https://app.netlify.com/sites/cheery-moxie-4f1121/deploys/6696725a7926eb00081ffb23
😎 Deploy Preview https://deploy-preview-369--cheery-moxie-4f1121.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Duddino Duddino self-assigned this May 20, 2024
@Duddino Duddino added the Enhancement New feature or request label May 20, 2024
@Duddino Duddino marked this pull request as ready for review May 21, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request. Burner address
1 participant