I absolutely, 100% support this issue.
It would speed up our processing when building our index by nearly double.
The shortcoming of eth_getLogs
came up recently in the Erigon issues (feature request: add field`timestamp` to `eth_getLogs` response · Issue #4951 · ledgerwatch/erigon · GitHub). For a short time, it was in eth_getLogs
, but it was moved to erigon_getLogs
because they reserve the eth_
namespace for “standards.” (In other words, there’s already an existing implementation.)
This is a simple win for every RPC user who uses logs and needs timestamps on those logs, as it immediately lessens the number of RPC queries by nearly half.
Such a change is also backwards compatible, since the timestamp
field didn’t exist in the past, adding it shouldn’t break anything (except possibly some test cases, but those can be easily updated).