rest: add interface for gettxspendingprevout rpc #33904

1. 
   kevkevinpal commented at 4:37 PM on November 18, 2025: contributor

   Summary

   This creates a REST interface for the gettxspendingprevout RPC, it also includes functional tests and updates to the REST docs

   This is part of #33808, and now that sstone’s PR #24539 is merged, I added mempool_only and return_spending_tx as query params.

2. DrahtBot added the label RPC/REST/ZMQ on Nov 18, 2025
3. 
   DrahtBot commented at 4:37 PM on November 18, 2025: contributor

   <!--e57a25ab6845829454e8d69fc972939a-->

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

   <!--006a51241073e994b41acfe9ec718e94-->

   Code Coverage & Benchmarks

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

   <!--021abf342d371248e50ceaed478a90ca-->

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   Concept ACK	sedited

   If your review is incorrectly listed, please copy-paste <code>&lt;!--meta-tag:bot-skip--&gt;</code> into the comment that the bot should ignore.

   <!--174a7506f384e20aa4161008e828411d-->

   Conflicts

   No conflicts as of last run.

   <!--5faf32d7da4f0f540f40219e4f7537a3-->

4. kevkevinpal force-pushed on Nov 18, 2025
5. DrahtBot added the label CI failed on Nov 18, 2025
6. DrahtBot removed the label CI failed on Nov 18, 2025
7. DrahtBot added the label Needs rebase on Dec 17, 2025
8. 
   fanquake commented at 7:21 AM on March 16, 2026: member

   would still need @sstone’s PR #24539, but that can be left as a follow-up for when it is merged.

   That PR has been merged.

9. kevkevinpal force-pushed on Mar 16, 2026
10. 
    kevkevinpal commented at 1:20 PM on March 16, 2026: contributor

    rebased to 9b4bc27

11. DrahtBot removed the label Needs rebase on Mar 16, 2026
12. kevkevinpal force-pushed on Mar 16, 2026
13. DrahtBot added the label CI failed on Mar 16, 2026
14. 
    kevkevinpal commented at 2:09 PM on March 16, 2026: contributor

    I updated to add mempool_only and return_spending_tx as query params

15. DrahtBot removed the label CI failed on Mar 16, 2026
16. sedited requested review from stickies-v on Mar 19, 2026
17. sedited requested review from stickies-v on Mar 19, 2026
18. 
    sedited commented at 9:42 AM on May 10, 2026: contributor

    Concept ACK

19. rest: add interface for gettxspendingprevout rpc

    This creates a REST interface for the gettxspendingprevout rpc and also
    includes functional tests

    84a1f671de
20. in src/rest.cpp:940 in 296b7ade11

     935 | +
     936 | +            for (size_t idx : unresolved) {
     937 | +                const COutPoint& prevout = prevouts[idx];
     938 | +                UniValue o(UniValue::VOBJ);
     939 | +                o.pushKV("txid", prevout.hash.ToString());
     940 | +                o.pushKV("vout", (uint64_t)prevout.n);
    

    ---

    ---

    

    maflcko commented at 8:07 AM on May 11, 2026:

    please no redundant and confusing c-style casts.

    Also, the new code should follow the code style for new code:

    • Clang-format-diff
    • multi-line ifs with {} braces
    • ...

    ---

    

    kevkevinpal commented at 1:55 PM on May 11, 2026:

    Thanks for the review! I pushed updates for the code style in 84a1f67

21. kevkevinpal force-pushed on May 11, 2026
22. 
    maflcko commented at 2:02 PM on May 11, 2026: member

    Also, please reword the pull description to remove @ mentions, which will turn into spam pings, when the text ends up in commit message(s)

23. 
    kevkevinpal commented at 2:06 PM on May 11, 2026: contributor

    Also, please reword the pull description to remove @ mentions, which will turn into spam pings, when the text ends up in commit message(s)

    Oops, good point. removed!

24. 
    kevkevinpal commented at 2:19 PM on May 11, 2026: contributor

    CI failure for Windows native, fuzz, VSWindows native, fuzz, VS is unrelated to the changes

25. DrahtBot added the label CI failed on May 11, 2026
26. DrahtBot removed the label CI failed on May 12, 2026

Contributors

kevkevinpalDrahtBotfanquakeseditedmaflcko

Review Requested
stickies-v