Skip to content
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

Document consensus rules from Zcash spec 7.1.2: Transaction Header #3425

Closed
11 tasks done
Tracked by #3125 ...
mpguerra opened this issue Jan 27, 2022 · 0 comments · Fixed by #3456
Closed
11 tasks done
Tracked by #3125 ...

Document consensus rules from Zcash spec 7.1.2: Transaction Header #3425

mpguerra opened this issue Jan 27, 2022 · 0 comments · Fixed by #3456
Assignees

Comments

@mpguerra
Copy link
Contributor

mpguerra commented Jan 27, 2022

  • The transaction version number MUST be greater than or equal to 1.
  • [Pre-Overwinter] The fOverwintered fag MUST NOT be set.
  • [Overwinter onward] The fOverwintered flag MUST be set.
  • [Overwinter onward] The version group ID MUST be recognized.
  • [Overwinter only, pre-Sapling] The transaction version number MUST be 3, and the version group ID MUST be 0x03C48270.
  • [Sapling to Canopy inclusive, pre-NU5] The transaction version number MUST be 4, and the version group ID MUST be 0x892F2085.
  • [NU5 onward] The transaction version number MUST be 4 or 5. If the transaction version number is 4 then the version group ID MUST be 0x892F2085. If the transaction version number is 5 then the version group ID MUST be 0x26A7270A.
  • [NU5 onward] The nConsensusBranchId field MUST match the consensus branch ID used for SIGHASH transaction hashes, as specified in [ZIP-244].
  • [Overwinter to Canopy inclusive, pre-NU5] nExpiryHeight MUST be less than or equal to 499999999.
  • [NU5 onward] nExpiryHeight MUST be less than or equal to 499999999 for non-coinbase transactions.
  • [Overwinter onward] If a transaction is not a coinbase transaction and its nExpiryHeight field is nonzero, then it MUST NOT be mined at a block height greater than its nExpiryHeight.
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 a pull request may close this issue.

2 participants