api3dao / airseeker-v1

Airseeker maintains beacons using signed data from an Airnode HTTP Signed Data Gateway
MIT License
2 stars 3 forks source link

Optimize undefined gas limit for individual Beacon updates #448

Open bbenligiray opened 1 year ago

bbenligiray commented 1 year ago

Blocked by https://github.com/api3dao/airseeker/issues/447 and https://github.com/api3dao/airseeker/issues/446

With self-funded Beacons, we also want to omit fulfillmentGasLimit (same reasons as https://github.com/api3dao/airseeker/issues/447), but we don't want to call estimateGas() for each update because it's wasteful (they will all return the same value). As a solution, in cases that fulfillmentGasLimit is not defined and the transaction to be made is a single Beacon update, we can use the estimateGas() result we got from https://github.com/api3dao/airseeker/issues/446

bbenligiray commented 1 year ago

@metobom Let's also drop this from Phase 2 to high priority Post-phase 2