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

Make ICS26Router pausable #231

Open
2 of 4 tasks
srdtrk opened this issue Jan 22, 2025 · 1 comment
Open
2 of 4 tasks

Make ICS26Router pausable #231

srdtrk opened this issue Jan 22, 2025 · 1 comment
Labels
needs discussion This issue needs more discussion before its implementation security This issue impacts security assumptions solidity Issues related to the solidity contracts type: feature Feature request

Comments

@srdtrk
Copy link
Member

srdtrk commented Jan 22, 2025

Summary

Allow a security council to pause ICS26Router. This pause should be reversible by the governance of the counterparty chain.

Use cases

  • Freezing the contract in case of a security vulnerability without being subject to a timelock that'd otherwise be enforced on the security council.
  • The freeze should be reversible by the governance of the counterparty chain which is also not subject to timelocks.

Proposal

Use openzepellin's Pausable contract

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
  • Estimate provided
@srdtrk srdtrk added solidity Issues related to the solidity contracts type: feature Feature request labels Jan 22, 2025
@srdtrk srdtrk added the security This issue impacts security assumptions label Jan 22, 2025
@srdtrk
Copy link
Member Author

srdtrk commented Jan 28, 2025 •
edited
Loading

I think this issue needs more discussion since it might make more sense to have only ICS20Transfer be pausable. Since we want to use ICA admin to unpause which would rely on ICS26Router to function

@srdtrk srdtrk added the needs discussion This issue needs more discussion before its implementation label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs discussion This issue needs more discussion before its implementation security This issue impacts security assumptions solidity Issues related to the solidity contracts type: feature Feature request
Projects
IBC-GO Eureka
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
1 participant
@srdtrk