Skip to content

feat: Persist contributed inputs#3

Merged
chavic merged 3 commits intomasterfrom
feat/Persist-contributed-inputs
Apr 12, 2026
Merged

feat: Persist contributed inputs#3
chavic merged 3 commits intomasterfrom
feat/Persist-contributed-inputs

Conversation

@ValeraFinebits
Copy link
Copy Markdown
Owner

Persist receiver input context across replayed payjoin states.

Store receiver-contributed input identity in session state and reuse it when replaying WantsFeeRange and ProvisionalProposal. Restrict proposal signing/finalization to receiver-contributed inputs, clear sender finalization and partial sigs, and validate that replayed proposals still contain the expected receiver inputs. Also tighten receiver change selection to fresh Change scripts, bound relay polling with a request timeout, and improve contribution failure diagnostics.

Persist receiver input context across replayed payjoin states
Store receiver-contributed input identity in session state and reuse it when replaying WantsFeeRange and ProvisionalProposal. Restrict proposal signing/finalization to receiver-contributed inputs, clear sender finalization and partial sigs, and validate that replayed proposals still contain the expected receiver inputs. Also tighten receiver change selection to fresh Change scripts, bound relay polling with a request timeout, and improve contribution failure diagnostics.
@ValeraFinebits ValeraFinebits marked this pull request as ready for review April 3, 2026 15:40
@ValeraFinebits ValeraFinebits requested a review from chavic April 3, 2026 15:40
Copy link
Copy Markdown
Collaborator

@chavic chavic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a useful improvement for replaying the in-memory receiver session.

@chavic chavic merged commit 846f177 into master Apr 12, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants