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.

Sign up for GitHub

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

Jump to bottom

feat(relayer): add quota manager, update relayer, requeue failed messages #17001

Merged
merged 64 commits into from
May 9, 2024
Merged

feat(relayer): add quota manager, update relayer, requeue failed messages #17001

merged 64 commits into from
May 9, 2024

Conversation

cyberhorsey
Copy link
Contributor

@cyberhorsey cyberhorsey commented May 6, 2024 •
edited
Loading

Also updates the watchdog service to pause both bridges, and adds metrics in case the pause fails, which we can use to alert.

dantaik and others added 30 commits May 3, 2024 12:10
@dantaik
add IRateLimiter
f08d003
@dantaik
Update ERC20Vault.sol
27b11cb
@dantaik
more
5e5d41c
@dantaik
Update RateLimiter.sol
9924c27
@dantaik
more
18f8d10
@dantaik
rename
d5d0f5f
@dantaik
Update RateLimiter.sol
458ae3a
@dantaik
Delete RateLimiter.sol
1c9539c
@dantaik
Merge branch 'main' into rate_limiter
89fe11c
@dantaik @adaki2004
Update packages/protocol/contracts/common/AddressResolver.sol
8de743e 
Co-authored-by: D <51912515+adaki2004@users.noreply.github.com>
@dantaik
Rate limiter impl (#16972)
b11bed0
@dantaik
Update QuotaManager.sol
e481721
@dantaik
rename
daf4d1a
@dantaik
rename
e62940e
@dantaik
Update ERC20Vault.sol
c08314f
@dantaik
Merge branch 'main' into rate_limiter
6702f92
@dantaik
more
b337bc7
@dantaik
Merge branch 'rate_limiter' of https://github.com/taikoxyz/taiko-mono …
df3dcec 
…into rate_limiter
@dantaik
Update Bridge.sol
919ec43
@dantaik
Update contract_layout.md
453d5f2
@dantaik
update
e0d9dca
@dantaik
more
0d197ef
@dantaik @github-actions
Add contract layout table
d7fd5d8
@dantaik
Merge branch 'main' into rate_limiter
eeb0a24
@dantaik @github-actions
Add contract layout table
55dea3b
@dantaik
Update QuotaManager.sol
9ed91b3
@dantaik
more
f9dd9d0
@dantaik
fix
acffccd
@dantaik
Create DeployL1QuotaManager.s.sol
be7e450
@dantaik
Update DeployL1QuotaManager.s.sol
1ef6dc2
@cyberhorsey cyberhorsey enabled auto-merge May 6, 2024 17:15
@cyberhorsey cyberhorsey added this pull request to the merge queue May 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch May 7, 2024
@dantaik dantaik self-requested a review May 7, 2024 03:09
@openzeppelin-code OpenZeppelin Code
Copy link

feat(relayer): add quota manager, update relayer, requeue failed messages

Generated at commit: d1b8f0e7ebf3f09e6f081fc08aa0f476bdeea9eb

🚨 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 added this pull request to the merge queue May 9, 2024
Merged via the queue into main with commit 90944a2 May 9, 2024
6 checks passed
@dantaik dantaik deleted the rate_limiter_relayer branch May 9, 2024 03:31
This was referenced May 9, 2024
Closed
Closed
@taiko-kitty taiko-kitty mentioned this pull request Jun 13, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidtaikocha davidtaikocha davidtaikocha approved these changes

@dantaik dantaik dantaik approved these changes

@xiaodino xiaodino Awaiting requested review from xiaodino

@RogerLamTd RogerLamTd Awaiting requested review from RogerLamTd

@k-kaddal k-kaddal Awaiting requested review from k-kaddal

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cyberhorsey @dantaik @davidtaikocha @xiaodino