BIP 322 Support

[trevormil]

As BIP322 support becomes more popular with Bitcoin wallets (Phantom, etc), it would be nice to have a Go implementation for message signature verification. To my knowledge, I could not find one. The main one I could find is https://github.com/ACken2/bip322-js.

I particularly need it for my project that I am working on. Is this something that can be completed anytime soon? Or should I find an alternative solution in the meantime?

[Roasbeef]

I particularly need it for my project that I am working on. Is this something that can be completed anytime soon? Or should I find an alternative solution in the meantime?

We'd accept a contribution implementing it. Seems relatively straight forward.

[mohamedawnallah]

I'd like to work on this issue, Thanks!

[trevormil]

I'd like to work on this issue, Thanks!

Awesome. Could you keep me updated on any progress?

[mohamedawnallah]

Okay, I will do :)

[trevormil]

Hey, any progress updates? My project will need a solution by the end of February. Do you think this will be done by then? Or, should I find an alternative?

No worries if not. Just need to know how to move forward.

[gamedevod]

@trevormil check my issue, i started implement it, but I warn you right away - my code does not produce correct results yet, and I cannot fix it https://github.com/btcsuite/btcd/issues/2117

[mohamedawnallah]

Sorry I seem missed this notification @trevormil I am busy now with other tasks Hope I get to it soon. That said please feel free to look for alternatives in the meantime. Thanks for your understanding 🙏

[trevormil]

Sorry I seem missed this notification @trevormil I am busy now with other tasks Hope I get to it soon. That said please feel free to look for alternatives in the meantime. Thanks for your understanding 🙏

No worries. Thank you.

[AbhinavMir]

@trevormil hi! I had a look at how bitcoin core does this and a quick gist would be:

• Adds script/proof.cpp and script/proof.h (for BIP 322 support).
• Updates wallet interfaces to include a new signMessage method for all address types.
• Modifies the dialog for signing/verifying messages to support BIP 322.
• Modify verifymessage for verbosity of BIP 322 details.
• Implement new logic in src/script/sign.cpp and src/script/sign.h for BIP 322 signature creation and verification.
• Update utility functions and test cases for BIP 322 message signing and verification compatibility.

I just got started on this and would be using this as a rough roadmap. A rather trivial question would be: Where does the files for bip322 go? 😅 The repo is HUGE and I'm new here.

[yemmyharry]

@trevormil @Roasbeef pls, I'd like to contribute to this. can I pick it up?

[trevormil]

@yemmyharry Yea, that would be great. I am not actively working on it. I kinda just settled with a workaround calling the JavaScript version from Go code. Could you keep me updated on progress?

[Roasbeef]

I just got started on this and would be using this as a rough roadmap. A rather trivial question would be: Where does the files for bip322 go? 😅 The repo is HUGE and I'm new here.

@AbhinavMir I think we'd add the bip322 code to the txscript package (where all the other Script logic lives). IIUC, BIP 322 is just a way to make a "fake transaction" to show that if an output were created with a given pkScript, you can spend it with a valid witness. The txscript package has all the contents (along with the wire) package needed to do such a construction.

@yemmyharry sure, feel free to use this issue to get feedback on any WIP code you may have.