Switch to getting logs via block hash instead of block number

[stwiname]

Description

Using the block height to get logs will always return results or empty results rather than an invalid result. Switching to using the block hash guarantees we get the right logs for the block. If an endpoint is out of sync and we're unable to get the logs then retry logic will handle it.

Fixes subquery#1999

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

Checklist

• I have tested locally
• I have performed a self review of my changes
• Updated any relevant documentation
• Linked to any relevant issues
• I have added tests relevant to my changes
• Any dependent changes have been merged and published in downstream modules
• My code is up to date with the base branch
• I have updated relevant changelogs. We suggest using chan

[jiqiang90]

Need benchmark for this change

[github-actions]

Coverage report

❌ An unexpected error occurred. For more details, check console

Error: The process '/usr/local/bin/yarn' failed with exit code 1

St.❔	Category	Percentage	Covered / Total
🔴	Statements	46.78%	3063/6547
🟡	Branches	64.34%	276/429
🔴	Functions	38.03%	89/234
🔴	Lines	46.78%	3063/6547

Test suite run failed

Failed tests: 3/41. Failed suites: 3/8.

  ● CeloJsonRpcProvider › should not set gasLimit to zero for blocks after the hard fork

    expect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 1

      Object {
    -   "hex": "0x01312d00",
    +   "hex": "0x00",
        "type": "BigNumber",
      }

      32 |       ]),
      33 |     );
    > 34 |     expect(BigNumber.from(block.gasLimit)).toEqual(BigNumber.from(0x1312d00));
         |                                            ^
      35 |   });
      36 | });
      37 |

      at Object.<anonymous> (packages/node/src/ethereum/ethers/celo/celo-json-rpc-batch-provider.spec.ts:34:44)

---

  ● CeloJsonRpcProvider › should not set gasLimit to zero for blocks after the hard fork

    expect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 1

      Object {
    -   "hex": "0x01312d00",
    +   "hex": "0x00",
        "type": "BigNumber",
      }

      32 |       ]),
      33 |     );
    > 34 |     expect(BigNumber.from(block.gasLimit)).toEqual(BigNumber.from(0x1312d00));
         |                                            ^
      35 |   });
      36 | });
      37 |

      at Object.<anonymous> (packages/node/src/ethereum/ethers/celo/celo-json-rpc-provider.spec.ts:34:44)

---

  ● CeloJsonRpcProvider › should not set gasLimit to zero for blocks after the hard fork

    expect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 1

      Object {
    -   "hex": "0x01312d00",
    +   "hex": "0x00",
        "type": "BigNumber",
      }

      32 |       ]),
      33 |     );
    > 34 |     expect(BigNumber.from(block.gasLimit)).toEqual(BigNumber.from(0x1312d00));
         |                                            ^
      35 |   });
      36 | });
      37 |

      at Object.<anonymous> (packages/node/src/ethereum/ethers/celo/celo-ws-provider.spec.ts:34:44)

Report generated by 🧪jest coverage report action from be66666

[stwiname]

This seems to fail with Alchemy and older blocks (e.g. 0x8243343df08b9751f5ca0c5f8c9c0460d8a9b6351066fae0acbd4d3e776de8bb) but works with newer blocks