Closed m-Peter closed 2 months ago
The recent changes enhance the blockchain API by adding a TransactionsRoot
field to the Block
structure, enabling better verification and integrity checks of transaction data. Alongside this, a new test assertion confirms that the retrieved block contains valid transactions, further strengthening the testing framework. Overall, these modifications ensure a more complete and reliable blockchain response.
File | Change Summary |
---|---|
api/api.go |
Added TransactionsRoot field to Block struct, improving block response data integrity. |
tests/web3js/eth_non_interactive_test.js |
Added assertion to validate that transactionsRoot in the block response is non-zero. |
Objective | Addressed | Explanation |
---|---|---|
Include the transactionsRoot hash in block response (#389) |
✅ |
🐰 "In blocks where hashes align,
A root of transactions, oh so fine!
With tests that check and validate,
Our blockchain's trust we celebrate!
Hopping 'round with joy so neat,
Integrity's our sweetest treat!" 🥕
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Closes: https://github.com/onflow/flow-evm-gateway/issues/389
Description
This field has now a non-zero value, so we should return the correct value in the related block APIs.
For contributor use:
master
branchFiles changed
in the Github PR explorerSummary by CodeRabbit
New Features
Bug Fixes