Skip to content

BIP375: match script_pubkey compliance with spec#79

Merged
tcharding merged 1 commit intorust-bitcoin:masterfrom
macgyver13:bip375-test-fix-bip370
Mar 19, 2026
Merged

BIP375: match script_pubkey compliance with spec#79
tcharding merged 1 commit intorust-bitcoin:masterfrom
macgyver13:bip375-test-fix-bip370

Conversation

@macgyver13
Copy link
Contributor

Previously, the MissingScriptPubkey check was entirely skipped when the silent-payments feature was enabled (#45, e1938eb). This was too broad — BIP375 only allows omitting PSBT_OUT_SCRIPT when a silent payment output PSBT_OUT_SP_V0_INFO is present.

Restore original bip370-parse-invalid validation behavior.

@macgyver13 macgyver13 marked this pull request as draft March 16, 2026 21:35
@macgyver13 macgyver13 force-pushed the bip375-test-fix-bip370 branch from 04a1853 to 9845ce0 Compare March 16, 2026 21:36
@macgyver13 macgyver13 marked this pull request as ready for review March 16, 2026 21:37
@tcharding
Copy link
Member

CI should be fixed now mate, can rebase.

Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

utACK 9845ce0

@macgyver13 macgyver13 force-pushed the bip375-test-fix-bip370 branch from 9845ce0 to 71444bd Compare March 18, 2026 13:28
Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

utACK 71444bd

@tcharding tcharding merged commit 4f6afdb into rust-bitcoin:master Mar 19, 2026
12 checks 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