feat: fix wrong cache key on eth_getBlockByHash

[natanasow]

Description:

The mainnet log shows incorrectness in the cache key.

Returning cached value eth_getBlockByHash_undefined_false:

{
   "timestamp":"0x672b8a43",
   "difficulty":"0x0",
   "extraData":"0x",
   "gasLimit":"0x1c9c380",
   "baseFeePerGas":"0x19077fd3000",
   "gasUsed":"0x128e00",
   "logsBloom":"0x00000000000000102001000000000000000000000000000100000000000000100000000000000000804000000000800000000000000020000000004040220000010040000000200800000000000000000000000200000001000208000000000000000000120000000040000000200800000000002000000000200040000800000000a00000000000000800200000100000000000000000000000000000000000020080000004000000000000000000000000000000000000000000000000000000082001000000000000000000000000000200000000000000000020000020000010000000012004000000000000000000000000000080100000000000000010",
   "miner":"0x0000000000000000000000000000000000000000",
   "mixHash":"0x0000000000000000000000000000000000000000000000000000000000000000",
   "nonce":"0x0000000000000000",
   "receiptsRoot":"0xcc412e69626441a403e43f13ec98131d07cd2447b5a14fedf73c944dabc6e166",
   "sha3Uncles":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347",
   "size":"0x954",
   "stateRoot":"0x56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421",
   "totalDifficulty":"0x0",
   "transactions":[
      "0xd6116fb5f24a499ea5f20fb610d3e2d9ae20c3a8c3c9832dc31e822bad1daf53"
   ],
   "transactionsRoot":"0x057c1438ce427869a7aee23fad82c8718ba275662dfcdbf9ce3d3c4594c170b0",
   "uncles":[
      
   ],
   "withdrawals":[
      
   ],
   "withdrawalsRoot":"0x0000000000000000000000000000000000000000000000000000000000000000",
   "number":"0x43ef5f9",
   "hash":"0x057c1438ce427869a7aee23fad82c8718ba275662dfcdbf9ce3d3c4594c170b0",
   "parentHash":"0x75a8fdf0df809c00a27de852177012330dc2f1946aaf5abcc1a7496f8a4d584b"
}

on eth_GetBlockByHash

Related issue(s):

Fixes #3231

Notes for reviewer:

Checklist

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

[github-actions]

Test Results

 17 files   -  2  230 suites   - 15   29m 48s ⏱️ - 1m 54s
607 tests  -  2  603 ✅ ± 0  4 💤 ±0  0 ❌  - 2 
623 runs   - 63  619 ✅  - 60  4 💤  - 1  0 ❌  - 2 

Results for commit cdf0020. ± Comparison against base commit acc4538.

This pull request removes 2 tests.

"before all" hook in "@precompile-calls Tests for eth_call with HTS" ‑ RPC Server Acceptance Tests Acceptance tests @precompile-calls Tests for eth_call with HTS "before all" hook in "@precompile-calls Tests for eth_call with HTS"
"before each" hook for "should execute "eth_getStorageAt" request to get old state with passing specific block" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before each" hook for "should execute "eth_getStorageAt" request to get old state with passing specific block"

♻️ This comment has been updated with latest results.

[ebadiere]

Can we add a test for this?

[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 SonarCloud

[ebadiere]

LG.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.88%. Comparing base (acc4538) to head (cdf0020).
Report is 23 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3232      +/-   ##
==========================================
- Coverage   83.17%   82.88%   -0.29%     
==========================================
  Files          66       69       +3     
  Lines        4314     4475     +161     
  Branches      843      881      +38     
==========================================
+ Hits         3588     3709     +121     
- Misses        483      497      +14     
- Partials      243      269      +26     

Flag	Coverage Δ
config-service	98.14% <ø> (ø)
relay	85.29% <100.00%> (ø)
server	83.52% <ø> (ø)
ws-server	36.87% <ø> (ø)

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

Files with missing lines	Coverage Δ
packages/relay/src/lib/eth.ts	83.68% <100.00%> (+1.41%)	⬆️

... and 12 files with indirect coverage changes

---- 🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles