search

llamanodes / web3-proxy

Fast loadbalancing and caching proxy for Ethereum or chains with similar JSON-RPC methods
https://llamanodes.com
GNU General Public License v3.0
153 stars 34 forks source link

Query window should start at a multiple #52

Closed yenicelik closed 1 year ago

yenicelik commented 1 year ago

Right now there is a small bug in that the very last window is not a multiple of "period". I should fix that

yenicelik commented 1 year ago

The last couple commits on this should fix it: https://github.com/yenicelik/web3-proxy/commit/7490688289e8247c07975593da5242090415611b

BlinkyStitt commented 1 year ago

Awesome. What was the cause?

that commit is part of https://github.com/llamanodes/web3-proxy/pull/44, right?

yenicelik commented 1 year ago

yes its part of #44. I'm not sure what caused it, but it's all handled by the influxdb query now (no arithmetic by me), and all outputs looked fine. I will close this for now let's re-open it when it re-occurs.