wallet: Add address validation in sendtoaddress RPC #33068

1. 
   w0xlt commented at 4:35 am on July 26, 2025: contributor

   Is there a specific reason not to validate the address in the sendtoaddress RPC? This PR adds that.
2. DrahtBot added the label Wallet on Jul 26, 2025
3. 
   DrahtBot commented at 4:35 am on July 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/33068.

   Reviews

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

4. wallet: Add address validation in `sendtoaddress` RPC bbfd55762f
5. w0xlt force-pushed on Jul 26, 2025
6. DrahtBot added the label CI failed on Jul 26, 2025
7. 
   DrahtBot commented at 4:41 am on July 26, 2025: contributor

   🚧 At least one of the CI tasks failed. Task previous releases, depends DEBUG: https://github.com/bitcoin/bitcoin/runs/46770879558 LLM reason (✨ experimental): The CI failed due to a syntax error in spend.cpp caused by a missing semicolon before a closing brace.

   Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:

   • Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

   • A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.

   • An intermittent issue.

   Leave a comment here, if you need help tracking down a confusing failure.

8. w0xlt closed this on Jul 26, 2025

   ---

Contributors
w0xlt DrahtBot

Labels
Wallet CI failed