Return error information on descriptor parse error #16123

pull Empact wants to merge 2 commits into bitcoin:master from Empact:descriptor-parse-error changing 6 files +113 −51
  1. Empact commented at 1:14 am on May 30, 2019: member
    This makes descriptors more forgiving by making them more communicative when they fail. These are user input and I expect informative error handling is likely to be appreciated.
  2. Return an error when Parse fails on a descriptor 89d6f052ce
  3. Return an error when ParsePubkey fails on a descriptor 7823b4e1c9
  4. fanquake added the label RPC/REST/ZMQ on May 30, 2019
  5. fanquake added the label Wallet on May 30, 2019
  6. DrahtBot commented at 4:19 am on May 30, 2019: member

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #15986 (Add unmodified-descriptor-with-checksum to getdescriptorinfo by sipa)
    • #15590 (Descriptor: add GetAddressType() and IsSegWit() by Sjors)
    • #15427 (Add support for descriptors to utxoupdatepsbt by sipa)

    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.

  7. Empact commented at 5:21 am on May 30, 2019: member
    Closing for now, in deference to the conflicting PRs
  8. Empact closed this on May 30, 2019

  9. 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: 2024-10-30 00:12 UTC

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