Issue #10542 Signmessage doesn't work with segwit addresses #12774

pull jakubtrnka wants to merge 2 commits into bitcoin:master from jakubtrnka:master changing 3 files +40 −4
  1. jakubtrnka commented at 10:22 PM on March 24, 2018: none

    bitcoin-cli verifymessage was extended to verify signatures against both bech32 and segwit-p2sh. Class PubkeyConsistencyVisitor has been added which takes extracted public key as constructor argument and verifies signatures against all meaningful CTxDestinations.

    Signmessage will be implemented in a next step.

  2. Issue #10542 Signmessage doesn't work with segwit addresses
    bitcoin-cli verifymessage was extended to verify signatures against both
    bech32 and segwit-p2sh. Class PubkeyConsistencyVisitor has been added
    which takes extracted public key as constructor argument and verifies
    signatures against all meaningful CTxDestinations.
    
    Signmessage will be implemented soon.
    5f96c4ca0e
  3. Issue #10542 Signmessage doesn't work with segwit addresses
    Tabs have been replaced by spaces
    37a4916aa5
  4. luke-jr commented at 11:14 PM on March 24, 2018: member

    NACK, it is by design that the deprecated message signature format is not supported for Segwit stuff.

    If you want signed messages, please work on coming up with a signature system that addresses the many shortcomings of the old style.

  5. fanquake added the label RPC/REST/ZMQ on Mar 24, 2018
  6. sipa commented at 11:25 PM on March 24, 2018: member

    @luke-jr I'm torn here.

    One the one hand you're right that we should aim to work on an improved standard that can deal with arbitrary scripts rather than just specific single-key addresses.

    On the other hand, it does seem like nobody is really working on a standard like that, and the continued lack of standard is doing users a disservice. At the same time it seems other software has already adopted extensions to the old signmessage format.

  7. luke-jr commented at 11:30 PM on March 24, 2018: member

    There's been quite a bit of discussion on the bitcoin-dev mailing list. I don't think it's fair to say nobody is working on it. And even if that were true, the solution would be for someone who wants signmessage, to begin working on it.

    I'm not sure how much of a disservice it is. I can't remember the last time I saw signmessage used for what it actually does - everyone who wants it, seems to be assuming things the signatures don't prove.

  8. kallewoof commented at 4:08 AM on March 26, 2018: member

    On the other hand, it does seem like nobody is really working on a standard like that, and the continued lack of standard is doing users a disservice. At the same time it seems other software has already adopted extensions to the old signmessage format.

    Discussion here: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2018-March/015818.html

  9. laanwj commented at 9:01 PM on March 29, 2018: member

    Closing this, as there is no agreement to do this.

  10. laanwj closed this on Mar 29, 2018

  11. DrahtBot locked this on Dec 16, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 15:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me