Skip to content

feat: add pause controller #117

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
Merged
merged 2 commits into from
Jun 10, 2025
Merged

feat: add pause controller #117

merged 2 commits into from
Jun 10, 2025

Conversation

zimpha
Copy link
Member

@zimpha zimpha commented Jun 5, 2025

This PR added a new contract PauseController to allow more fine-grained access control over the setPause functions in L1ScrollMessenger, ScrollChain and EnforcedTxGateway.

This PauseController also added a pauseCooldownPeriod to prevent consecutive pause right after unpause. The owner of the PauseController should be ScrollOwner in production and the access of pause and unpause are configured in ScrollOwner.

Thegaram
Thegaram reviewed Jun 5, 2025
View reviewed changes
@zimpha zimpha merged commit 598d630 into main Jun 10, 2025
5 checks passed
@zimpha zimpha deleted the feat/pause_controller branch June 10, 2025 10:27
Thegaram pushed a commit that referenced this pull request Jun 26, 2025
@zimpha @Thegaram
feat: add pause controller (#117)
b0c7170
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonastheis jonastheis jonastheis left review comments

@Thegaram Thegaram Thegaram approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@zimpha @jonastheis @Thegaram