Skip to content

fix: include aa_authorization_list in the 0x76/0x78 signing-hash preimages #696

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.

Sign up for GitHub

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

Open
compusophy wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: include aa_authorization_list in the 0x76/0x78 signing-hash preimages #696

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 10 additions & 5 deletions src/pages/docs/protocol/transactions/spec-tempo-transaction.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -323,7 +323,9 @@ sender_hash = keccak256(0x76 || rlp([
valid_before,
valid_after,
0x80, // fee_token encoded as EMPTY (skipped)
0x00 // placeholder byte for fee_payer_signature
0x00, // placeholder byte for fee_payer_signature
aa_authorization_list,
key_authorization? // Only encoded if present (backwards compatible)
]))

// When no fee_payer_signature:
Expand All @@ -339,7 +341,9 @@ sender_hash = keccak256(0x76 || rlp([
valid_before,
valid_after,
fee_token, // fee_token is INCLUDED
0x80 // empty for no fee_payer_signature
0x80, // empty for no fee_payer_signature
aa_authorization_list,
key_authorization? // Only encoded if present (backwards compatible)
]))
```

Expand All @@ -364,9 +368,10 @@ fee_payer_hash = keccak256(0x78 || rlp([ // Note: 0x78 magic byte
nonce,
valid_before,
valid_after,
fee_token, // fee_token ALWAYS included
sender_address // 20-byte sender address
key_authorization,
fee_token, // fee_token ALWAYS included
sender_address, // 20-byte sender address
aa_authorization_list,
key_authorization? // Only encoded if present (backwards compatible)
]))
```

Expand Down