-
Couldn't load subscription status.
- Fork 88
feat: update getBlockReceipts to return null when block is not found
#3833
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.
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
feat: update getBlockReceipts to return null when block is not found
#3833
Conversation
…und (hiero-ledger#3813) Signed-off-by: Michał Walczak <[email protected]>
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May we add an acceptance test in the respective batch as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
…ansactionReceipt` (hiero-ledger#3813) Signed-off-by: Michał Walczak <[email protected]>
…s not found (hiero-ledger#3813) Signed-off-by: Michał Walczak <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov ReportAttention: Patch coverage is
@@ Coverage Diff @@
## main #3833 +/- ##
=======================================
Coverage ? 47.56%
=======================================
Files ? 83
Lines ? 4770
Branches ? 978
=======================================
Hits ? 2269
Misses ? 2163
Partials ? 338
🚀 New features to boost your workflow:
|
|
Hi @konstantinabl |
…und (hiero-ledger#3833) Signed-off-by: Michał Walczak <[email protected]> Co-authored-by: konstantinabl <[email protected]>
Description:
When querying the eth_getBlockReceipts method in Hedera’s JSON-RPC API with a valid but nonexistent block hash, the endpoint throws an error. This behavior is inconsistent with Ethereum-compatible JSON-RPC endpoints (such as Sepolia), which return null for nonexistent blocks, as per the standard JSON-RPC specification.
Related issue(s):
Fixes #3813
Checklist