rpc: correct `getnewaddress`/`getrawchangeaddress` address_type helptext #23676

pull brianddk wants to merge 1 commits into bitcoin:master from brianddk:rpc_bech32m_doc_update changing 1 files +2 −2
  1. brianddk commented at 12:53 AM on December 5, 2021: contributor

    RPC calls getnewaddress/getrawchangeaddress support the address_type of bech32m but it is omitted in the RPCHelpMan help text.

    The createmultisig and addmultisigaddress help text was not updated since bech32m is not yet supported in these.

  2. correct rpc address_type helptext
    added address_type of `bech32m` to rpc calls `getnewaddress`/`getrawchangeaddress`
    5767208504
  3. DrahtBot added the label RPC/REST/ZMQ on Dec 5, 2021
  4. DrahtBot added the label Wallet on Dec 5, 2021
  5. DrahtBot commented at 2:57 AM on December 5, 2021: member

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #23667 (Split up rpcwallet by meshcollider)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  6. shaavan commented at 1:40 PM on December 5, 2021: contributor

    Concept ACK

    I agree with the updates done in this PR.

    However, I tried to verify the following statement:

    The createmultisig and addmultisigaddress help text was not updated since bech32m is not yet supported in these.

    But I was not able to find any trusted material. It would help me correctly review the PR if you shared some article explaining this statement. Thank You.

  7. brianddk commented at 11:22 PM on December 5, 2021: contributor

    @shaavan , there is exception handling in both createmultisig and addmultisigaddress that will explicitly fail the command if an address_type of bech32m is provided.

    References:

  8. shaavan approved
  9. shaavan commented at 11:21 AM on December 6, 2021: contributor

    ACK 576720850467b7b21ca1ab59deab27b7a0c1c176

    Thanks for the references, @brianddk.

    I have checked that the changes made in this PR are correct and complete.

  10. MarcoFalke merged this on Dec 7, 2021
  11. MarcoFalke closed this on Dec 7, 2021

  12. sidhujag referenced this in commit 102a99c730 on Dec 7, 2021
  13. RandyMcMillan referenced this in commit d64c6d501b on Dec 23, 2021
  14. DrahtBot locked this on Dec 7, 2022

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:14 UTC

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