Skip to content

fix(sdk-coin-tempo): fix nonce 0 RLP encoding and verifyTransaction#8263

Merged
nayandas190 merged 1 commit intomasterfrom
cecho-325
Mar 8, 2026
Merged

fix(sdk-coin-tempo): fix nonce 0 RLP encoding and verifyTransaction#8263
nayandas190 merged 1 commit intomasterfrom
cecho-325

Conversation

@nayandas190
Copy link
Contributor

@nayandas190 nayandas190 commented Mar 8, 2026

Ticket: CECHO-325

  • ethers.utils.hexlify(0) emits "0x00" but Tempo's RLP decoder requires empty bytes ("0x") for zero, breaking first withdrawals (nonce 0). Use bigintToHex which handles this correctly.
  • Add 0x-prefix normalization in verifyTransaction for TSS signableHex compatibility. Guard parseTransaction against non-TIP-20 hex.

@nayandas190 nayandas190 marked this pull request as ready for review March 8, 2026 07:42
@nayandas190 nayandas190 requested a review from a team as a code owner March 8, 2026 07:42
@nayandas190 nayandas190 merged commit 5b21168 into master Mar 8, 2026
21 checks passed
@nayandas190 nayandas190 deleted the cecho-325 branch March 8, 2026 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nvrakesh06 nvrakesh06 nvrakesh06 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nayandas190 @nvrakesh06