Skip to content

fix: use consistent rlp encoding for KeyAuthorization#40

Open
decofe wants to merge 2 commits intomainfrom
klkvr/fix-rlp-encoding
Open

fix: use consistent rlp encoding for KeyAuthorization#40
decofe wants to merge 2 commits intomainfrom
klkvr/fix-rlp-encoding

Conversation

@decofe
Copy link
Copy Markdown
Member

@decofe decofe commented Apr 2, 2026

Matches the encoding change in tempoxyz/tempo#3443. expiry is no longer encoded as 0x80 when trailing (no limits present). Treats expiry=0 the same as expiry=None.

Prompted by: klkvr

Co-authored-by: Arsenii Kulikov <62447812+klkvr@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4f6f-abd8-73c6-8b2e-e785405a884a
decofe added a commit that referenced this pull request Apr 3, 2026
- Sync base ABI from tempo-std via tempo_abis.sh --sync
- Add T3 authorizeKey overload + new T3 functions on top (tempo-std
  not yet updated for T3)
- Resolve cherry-pick conflict with PR #40 RLP fix (expiry=0 handling
  already in as_rlp_payload)
- Fix test_inline_key_auth_with_limits: pass SignedKeyAuthorization
  object instead of raw bytes

Co-Authored-By: grandizzy <38490174+grandizzy@users.noreply.github.com>
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