-
Notifications
You must be signed in to change notification settings - Fork 38.8k
Silent Payments: send and receive #27827
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Silent Payments: send and receive #27827
Conversation
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. ConflictsReviewers, this pull request conflicts with the following ones:
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. |
ef70044 to
ecc8b79
Compare
ecc8b79 to
9f75c2e
Compare
Co-authored-by: Aurèle Oulès <[email protected]>
Co-authored-by: josibake <[email protected]>
106d8b5 to
a04d635
Compare
|
Maybe add a quick summary in the description with the main implementation differences relative to #24897. It seems a big one is that this doesn't require an index! You should also link from the child PRs back to this one. |
|
🐙 This pull request conflicts with the target branch and needs rebase. |
For reviewers
In an attempt to make reviewing a bit more sane, I'm breaking this up into a few smaller PRs, but will keep this one open as the parent PR and keep it rebased on the child PRs. The main purpose of having this PR is to track progress on child PRs and also have an easy way to compile
bitcoindwith both send and receive support for testing. Additionally, I'll be adding more functional tests to this PR since it's much easier to test whenbitcoindcan both send and receive.PRs
For the silent payments specification, see bitcoin/bips#1458
Overall
This PR implements the full silent payments scheme: sending and receiving. The following items are not covered in this PR and are intended for follow-up PRs:
sendtoaddressandsendmanyare covered in this PR)Major changes
This PR is a continuation of the work done in #24897. Below is a summary of the major changes:
rev*.datfiles for rescanning. It may make sense to add an index in the future, but for the purpose of vending tweak data to light clients, which is still an open questionIt may be helpful for context to read through the discussions on #24897 , but ongoing review should happen in the relevant child PRs listed above.