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

chore(protocol): lower L2-to-L1 bridge quota #17020

Merged
merged 13 commits into from
May 8, 2024
Merged

Conversation

dantaik
Copy link
Contributor

@dantaik dantaik commented May 7, 2024

Regarding the pr, I see you discuss timeframe and amount, if it's helpful I ran a quick dune query to see how it evolved for Linea. Their current limit is 18_750 ether but they started fairly small, and the time period is 24 hours (not changeable). Arguably it wasn't a bull market at the time, the price of ETH has changed and it's not Taiko, but gives an order of magnitude

It seems Linea's current ETH quota is 18_750ETH per 24 hours, or 195.3125ETH per 15 minutes. Therefore, I think we can lower our quota as well.

Copy link

openzeppelin-code bot commented May 7, 2024

chore(protocol): lower ETH L2->L1 quota

Generated at commit: 60b1f36fb97982c43e8cf310133e28c379e81028

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total
2
2
0
5
41
50
Dependencies Critical
High
Medium
Low
Note
Total
0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

@dantaik dantaik marked this pull request as ready for review May 7, 2024 11:01
@dantaik dantaik changed the title chore(protocol): lower ETH L2->L1 quota chore(protocol): lower L2->L1 bridge quota May 7, 2024
@dantaik dantaik changed the title chore(protocol): lower L2->L1 bridge quota chore(protocol): lower L2-to-L1 bridge quota May 7, 2024
@davidtaikocha davidtaikocha added this pull request to the merge queue May 8, 2024
Merged via the queue into main with commit 6ee06f1 May 8, 2024
4 checks passed
@davidtaikocha davidtaikocha deleted the change_ether_quota branch May 8, 2024 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants