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

Use independent packet type for unbonding information instead of ack, and put in same ordered channel as slashing packets #39

Closed
jtremback opened this issue Feb 9, 2022 · 0 comments · Fixed by #188
Closed

Use independent packet type for unbonding information instead of ack, and put in same ordered channel as slashing packets #39

jtremback opened this issue Feb 9, 2022 · 0 comments · Fixed by #188
Assignees
@mpoke
Labels
type: bug Issues that need priority attention -- something isn't working

Comments

@jtremback
Copy link
Contributor

This will prevent slashing and unbonding packets getting out of order which could stop necessary slashings from happening.
It will also mean that we don’t need to store every packet for the whole unbonding period.
@mpoke mpoke moved this to Todo in Replicated Security Apr 11, 2022
@mpoke mpoke added type: feature-request New feature or request improvement spec labels Apr 26, 2022
@mpoke mpoke self-assigned this Apr 29, 2022
@mpoke mpoke added type: bug Issues that need priority attention -- something isn't working and removed type: feature-request New feature or request improvement labels May 10, 2022
@mpoke mpoke moved this from Todo to Next in Replicated Security May 11, 2022
@mpoke mpoke moved this from Next to Todo in Replicated Security May 13, 2022
This was referenced Jun 6, 2022
Closed
Merged
This was referenced Jun 8, 2022
Closed
Closed
Closed
@mpoke mpoke added the mvcc label Jun 8, 2022
@mpoke mpoke moved this from Todo to Next in Replicated Security Jun 8, 2022
@mpoke mpoke moved this from Next to In Progress in Replicated Security Jun 24, 2022
@mpoke mpoke mentioned this issue Jun 29, 2022
Merged
@mpoke mpoke moved this from In Progress to Waiting for review in Replicated Security Jun 29, 2022
@mpoke mpoke closed this as completed in #188 Jul 7, 2022
Repository owner moved this from Waiting for review to Done in Replicated Security Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mpoke mpoke

Labels
type: bug Issues that need priority attention -- something isn't working
Projects
No open projects
Replicated Security
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add VSCMatured packets instead of acks cosmos/interchain-security
3 participants
@jtremback @mpoke @sainoe