feat: enhanced eth_getLogs with timestamp range validation and new error handling

[quiet-node]

Description:
This pull request improves the eth_getLogs functionality by:

• Introducing a new error: TIMESTAMP_RANGE_TOO_LARGE for handling large timestamp ranges.
• Adding validation to ensure the timestamp range between fromBlock and toBlock is within acceptable limits, 7 days (604800 seconds). (Fixes [eth_getLogs Failure] High Failure Rate with eth_getLogs Requests #3428)
• Ensuring eth_getLogs gracefully handles cases where toBlock is not provided. Without toBlock, the params.lte field is omitted, leading to request rejected by MN. To address this, the function now returns an empty response to the client. (Fixes [eth_getLogs Failure] Handle non-existent toBlock in validateBlockRangeAndAddTimestampToParams method #3430)

Tests and comments have been added to support and clarify these changes.

Related issue(s):

Fixes #3428

Notes for reviewer:

Checklist

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

[github-actions]

Test Results

 18 files   -   4  236 suites   - 35   33m 5s ⏱️ - 32m 38s
613 tests +  4  609 ✅ + 17  4 💤 +1  0 ❌  - 14 
629 runs   - 217  625 ✅  - 199  4 💤 ±0  0 ❌  - 18 

Results for commit 6b90c41. ± Comparison against base commit 373f1dc.

This pull request removes 4 and adds 8 tests. Note that renamed tests count towards both.

"after all" hook in "RPC Server Acceptance Tests" ‑ RPC Server Acceptance Tests "after all" hook in "RPC Server Acceptance Tests"
"before all" hook for "should execute "eth_getCode" for hts token" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode "before all" hook for "should execute "eth_getCode" for hts token"
"before all" hook in "@api-batch-2 RPC Server Acceptance Tests" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before all" hook in "@api-batch-2 RPC Server Acceptance Tests"
"before each" hook for "from/to Addresses when transferring HTS tokens to the tokenAddress are in evm and long-zero format" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before each" hook for "from/to Addresses when transferring HTS tokens to the tokenAddress are in evm and long-zero format"

@release should execute "eth_getCode" for contract evm_address ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode @release should execute "eth_getCode" for contract evm_address
@release should execute "eth_getCode" for contract with id converted to evm_address ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode @release should execute "eth_getCode" for contract with id converted to evm_address
should execute "eth_getCode" for hts token ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode should execute "eth_getCode" for hts token
should not return contract bytecode after sefldestruct ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode should not return contract bytecode after sefldestruct
should return 0x0 for account alias on eth_getCode ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode should return 0x0 for account alias on eth_getCode
should return 0x0 for account evm_address on eth_getCode ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode should return 0x0 for account evm_address on eth_getCode
should return 0x0 for non-existing contract on eth_getCode ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests eth_getCode should return 0x0 for non-existing contract on eth_getCode
should return empty logs if `toBlock` is not found ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-1 RPC Server Acceptance Tests RPC Server Acceptance Tests eth_getLogs should return empty logs if `toBlock` is not found

♻️ This comment has been updated with latest results.

[Nana-EC]

Looking good some questions

[natanasow]

LG, some nits that can be addressed in a follow-up PR due to the urgency of this one.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[konstantinabl]

LGTM, we just have to discuss if we will limit users to 7 days or we somehow handle this limiti from the MN side

[codecov]

Codecov Report

Attention: Patch coverage is 91.66667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 85.35%. Comparing base (373f1dc) to head (6b90c41).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
.../lib/services/ethService/ethCommonService/index.ts	94.11%	0 Missing and 2 partials ⚠️
.../lib/services/ethService/ethFilterService/index.ts	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3431      +/-   ##
==========================================
+ Coverage   84.18%   85.35%   +1.16%     
==========================================
  Files          69       69              
  Lines        4711     4738      +27     
  Branches     1048     1056       +8     
==========================================
+ Hits         3966     4044      +78     
+ Misses        428      394      -34     
+ Partials      317      300      -17     

Flag	Coverage Δ
config-service	98.14% <ø> (ø)
relay	79.26% <91.66%> (+0.04%)	⬆️
server	83.30% <ø> (ø)
ws-server	36.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
packages/relay/src/lib/errors/JsonRpcError.ts	75.00% <100.00%> (+0.49%)	⬆️
packages/relay/src/lib/eth.ts	86.11% <ø> (+4.84%)	⬆️
.../lib/services/ethService/ethFilterService/index.ts	80.43% <0.00%> (-2.18%)	⬇️
.../lib/services/ethService/ethCommonService/index.ts	91.46% <94.11%> (+0.15%)	⬆️

... and 4 files with indirect coverage changes