Skip to content

Conversation

@mwb-al
Copy link
Contributor

@mwb-al mwb-al commented Jun 10, 2025

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

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@lfdt-bot
Copy link

lfdt-bot commented Jun 10, 2025

🎉 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)

@mwb-al mwb-al marked this pull request as ready for review June 10, 2025 09:02
@mwb-al mwb-al requested review from a team as code owners June 10, 2025 09:02
@mwb-al mwb-al requested a review from konstantinabl June 10, 2025 09:02
@quiet-node quiet-node added the enhancement New feature or request label Jun 10, 2025
@quiet-node quiet-node added this to the 0.69.0 milestone Jun 10, 2025
Copy link
Contributor

@konstantinabl konstantinabl left a 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?

acuarica
acuarica previously approved these changes Jun 11, 2025
Copy link
Contributor

@acuarica acuarica left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@mwb-al mwb-al requested a review from konstantinabl June 12, 2025 11:22
@natanasow natanasow modified the milestones: 0.69.0, 0.70.0 Jun 13, 2025
Copy link
Contributor

@simzzz simzzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@quiet-node quiet-node left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Jun 17, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...b/services/ethService/blockService/BlockService.ts 0.00% 1 Missing ⚠️
@@           Coverage Diff           @@
##             main    #3833   +/-   ##
=======================================
  Coverage        ?   47.56%           
=======================================
  Files           ?       83           
  Lines           ?     4770           
  Branches        ?      978           
=======================================
  Hits            ?     2269           
  Misses          ?     2163           
  Partials        ?      338           
Files with missing lines Coverage Δ
packages/relay/src/index.ts 100.00% <ø> (ø)
packages/relay/src/lib/eth.ts 51.76% <ø> (ø)
...b/services/ethService/blockService/BlockService.ts 31.53% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mwb-al
Copy link
Contributor Author

mwb-al commented Jun 17, 2025

Hi @konstantinabl
I’ve added it. Could you please approve this PR?

@arianejasuwienas arianejasuwienas merged commit 27bcc1c into hiero-ledger:main Jun 18, 2025
42 of 43 checks passed
mwb-al added a commit to mwb-al/hiero-json-rpc-relay that referenced this pull request Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

eth_getBlockReceipts returns an error instead of null for unknown block hashes

8 participants