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

Chain not halting on broken invariant asserted with MsgVerifyInvariant #15325

Closed
ba1ciu opened this issue Mar 9, 2023 · 1 comment
Closed

Chain not halting on broken invariant asserted with MsgVerifyInvariant #15325

ba1ciu opened this issue Mar 9, 2023 · 1 comment
Labels
C:x/crisis

Comments

@ba1ciu
Copy link

ba1ciu commented Mar 9, 2023

Summary of Bug

Panics made by VerifyInvariant keeper function are recovered at the middleware level. Is this an expected behaviour?

Version

v0.47.0-rc2
@github-actions github-actions bot added the needs-triage Issue that needs to be triaged label Mar 9, 2023
@tac0turtle tac0turtle added C:x/crisis and removed needs-triage Issue that needs to be triaged labels Mar 20, 2023
@tac0turtle
Copy link
Member

hey we released a small advisory about this, you are correct it doesnt error out. We have a open epic about this here #15706

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C:x/crisis
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tac0turtle @ba1ciu