Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug in optimism gas oracle #336

Open
seunlanlege opened this issue Nov 5, 2024 · 3 comments · May be fixed by #352
Open

Fix bug in optimism gas oracle #336

seunlanlege opened this issue Nov 5, 2024 · 3 comments · May be fixed by #352
Labels
good first issue Good for newcomers

Comments

@seunlanlege
Copy link
Member

seunlanlege commented Nov 5, 2024

The Tesseract relayer has a sub module referred to as the gas Oracle. This module is responsible for estimating the gas price, gas cost, and USD equivalent for cross chain transactions.

It is currently broken for the optimism chain, ever since the migration to blobs it has been broken and it over estimates the gas cost,

Which is apparently now a multidimensional value comprised of the L1 blob fee and gas costs as well as the L2 execution fee.

In order for relayers to correctly estimate the costs of the optimism transactions. This estimation process must be investigated and fixed.

@seunlanlege seunlanlege added the good first issue Good for newcomers label Nov 5, 2024
@mahmudsudo
Copy link

Hi can I take on this ?

@seunlanlege
Copy link
Member Author

Hi can I take on this ?

Yeah definitely!

@sylvaincormier
Copy link

Hi! Since it appears there hasn't been any progress on this issue, I'd like to take it on.

sylvaincormier added a commit to sylvaincormier/hyperbridge that referenced this issue Dec 14, 2024
- Add OptimismGasComponents struct to track L1/L2 fees
- Implement proper L1 data posting cost calculation
- Consider both base fee and blob fee with scalars
- Fix gas price overestimation issue

Fixes polytope-labs#336
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants