docs: sync external-signer.md with current external signer flow #33947

pull GarmashAlex wants to merge 1 commits into bitcoin:master from GarmashAlex:docs/external-signer-sync-stdin-signtx changing 1 files +21 −22
  1. GarmashAlex commented at 9:45 am on November 26, 2025: none
    Updated external-signer.md to match the current implementation. The signer network selection is passed as –chain rather than –testnet, and transaction signing is performed via stdin using the signtx command with JSON output containing a psbt field or error. Wallet creation for external signers requires external_signer=true with private keys disabled, so the example now uses named arguments. Spending with an external signer goes through the new send/sendall RPCs which build a PSBT, call the signer, and finalize when signatures are sufficient; sendtoaddress is not applicable for such wallets and also returns a plain txid for pk-enabled wallets rather than a complete field. The getdescriptors call is documented with the –account parameter and return shape including receive and internal arrays. Examples for displayaddress and createwallet are corrected, enumeratesigners now shows an optional name field, and references to bitcoin-qt replace the non-existent “bitcoin node” and “bitcoin rpc”.
  2. docs: sync external-signer.md with current external signer flow d7674c68e0
  3. DrahtBot added the label Docs on Nov 26, 2025
  4. DrahtBot commented at 9:45 am on November 26, 2025: contributor

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

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33947.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  5. fanquake commented at 9:48 am on November 26, 2025: member
    Thanks, however there is an existing PR at #33765. I’d suggest reviewing there first.
  6. fanquake closed this on Nov 26, 2025

  7. GarmashAlex commented at 12:20 pm on November 26, 2025: none

    Thanks, however there is an existing PR at #33765. I’d suggest reviewing there first.

    Oh, actually didn’t notice that PR


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: 2025-11-27 00:13 UTC

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