RPC: Return external_signer in getwalletinfo #24307

1. 
   kristapsk commented at 1:27 am on February 10, 2022: contributor

   Add external_signer to the result object of getwalletinfo RPC which indicates whether WALLET_FLAG_EXTERNAL_SIGNER flag is set for the wallet.
2. RPC: Return external_signer in getwalletinfo b75f4c89ec
3. DrahtBot added the label RPC/REST/ZMQ on Feb 10, 2022
4. DrahtBot added the label Wallet on Feb 10, 2022
5. unknown approved
6. 
   unknown commented at 3:28 am on February 10, 2022: none

   utACK https://github.com/bitcoin/bitcoin/pull/24307/commits/b75f4c89ec4d33a3014ccd5151964881b5e0aa1c
7. 
   S3RK commented at 7:12 am on February 10, 2022: member

   utACK b75f4c89ec4d33a3014ccd5151964881b5e0aa1c
8. 
   Sjors commented at 6:09 pm on February 10, 2022: member

   #21928 contains this change as well, though it’s valid on its own.
9. 
   ghost commented at 6:19 pm on February 10, 2022: none

   #21928 contains this change as well, though it’s valid on its own. @Sjors why not build on top of this PR?

10. 
    Sjors commented at 6:33 pm on February 10, 2022: member

    @prayank23 my PR is almost a year old. But if this gets merged first I’ll rebase it of course.
11. 
    kristapsk commented at 9:22 pm on February 10, 2022: contributor

    @Sjors Didn’t know about your PR. Was playing around with external signer stuff and at one moment wanted to check is that flag really set for a specific wallet and noticed I can’t do that. Anyway, this one is simple change, should be easy to review, test and merge sooner.
12. 
    ghost commented at 9:28 pm on February 10, 2022: none

    @prayank23 my PR is almost a year old. But if this gets merged first I’ll rebase it of course.

    TBH I can’t review all the things in #21928

    This looks ready for review and merge :)

13. fanquake requested review from achow101 on Feb 11, 2022
14. 
    DrahtBot commented at 11:36 am on February 11, 2022: 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:

    • #21928 (wallet: allow toggling external_signer flag by Sjors)

    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.

15. 
    brunoerg commented at 1:30 pm on February 11, 2022: member

    utACK b75f4c89ec4d33a3014ccd5151964881b5e0aa1c
16. 
    achow101 commented at 5:19 pm on February 11, 2022: member

    ACK b75f4c89ec4d33a3014ccd5151964881b5e0aa1c
17. achow101 merged this on Feb 11, 2022
18. achow101 closed this on Feb 11, 2022

    ---

19. kristapsk deleted the branch on Feb 11, 2022
20. sidhujag referenced this in commit e80e1e1a33 on Feb 12, 2022
21. 
    luke-jr commented at 11:37 pm on February 12, 2022: member

    Seems like this should probably be a string indicating the external signer program… but I guess changing it later is okay?
22. 
    Sjors commented at 7:17 pm on February 15, 2022: member

    Fine by me :-) I’ll rebase. @luke-jr that would just echo -signer though
23. 
    luke-jr commented at 8:01 pm on February 15, 2022: member

    For now, yes - but eventually it’d be ideal to have wallets with different external signers.
24. 
    kristapsk commented at 8:33 pm on February 15, 2022: contributor

    I agree with @luke-jr, different external signers for different wallets in the future makes sense.
25. 
    Sjors commented at 9:11 pm on February 15, 2022: member

    different external signers

    To clarify: different external signer programs, e.g. alternatives to HWI. You can already have multiple wallets each using a different (type of) physical device, all via HWI.

26. 
    w0xlt commented at 11:19 pm on February 16, 2022: contributor

    ACK b75f4c8
27. DrahtBot locked this on Feb 16, 2023

Contributors
kristapsk S3RK Sjors ghost DrahtBot brunoerg achow101 luke-jr w0xlt

Review Requested
achow101

Labels
Wallet RPC/REST/ZMQ