Skip to content

Enforce minimal RLP integer decoding#351

Merged
q9f merged 1 commit intomainfrom
eg3t6w-codex/add-deserialize_rlp_int-and-update-transaction-decodes
Aug 4, 2025
Merged

Enforce minimal RLP integer decoding#351
q9f merged 1 commit intomainfrom
eg3t6w-codex/add-deserialize_rlp_int-and-update-transaction-decodes

Conversation

@q9f
Copy link
Owner

@q9f q9f commented Aug 4, 2025

  • add deserialize_rlp_int helper to delegate to Rlp::Sedes.big_endian_int.deserialize
  • require minimal integer encoding across all transaction decoders
  • test for RLP integer minimality and round-trip safety

fix #291

@q9f q9f added the codex Codex is a durable, decentralised data storage protocol label Aug 4, 2025 — with ChatGPT Codex Connector
@codecov
Copy link

codecov bot commented Aug 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@895f78e). Learn more about missing BASE report.
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #351   +/-   ##
=======================================
  Coverage        ?   99.08%           
=======================================
  Files           ?       85           
  Lines           ?     5766           
  Branches        ?        0           
=======================================
  Hits            ?     5713           
  Misses          ?       53           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@q9f q9f merged commit 1dce05c into main Aug 4, 2025
8 checks passed
@q9f q9f deleted the eg3t6w-codex/add-deserialize_rlp_int-and-update-transaction-decodes branch August 4, 2025 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex Codex is a durable, decentralised data storage protocol

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should transaction decoding permit leading \x00?

1 participant