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.

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

Implement packet delay in packet relaying #640

Closed
3 of 5 tasks
ancazamfir opened this issue Feb 11, 2021 · 0 comments · Fixed by #747
Closed
3 of 5 tasks

Implement packet delay in packet relaying #640

ancazamfir opened this issue Feb 11, 2021 · 0 comments · Fixed by #747
Assignees
Labels
O: security Objective: cause to enhance security and improve safety
Milestone

Comments

@ancazamfir
Copy link
Collaborator

ancazamfir commented Feb 11, 2021

Crate

ibc-relayer and ibc-relayer-cli

Summary

To prevent certain attacks it is required that there is a delay between MsgUpdateClient and any packet messages: MsgRecvPacket, MsgAcknowledgment, MsgTimeout. This requires changes in the relayer that currently sends the Update Client in the same transaction as the packets. The lag should be obtained from the connection delay period parameter. Currently this is ignored if set by other relayer, always set to 0 by this relayer.

See cosmos/ibc#507 for more details

Note: This concerns the relayer functionality.
We'll cover packet delays in the IBC module in a separate issue.

Proposal

Will be detailed soon


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O: security Objective: cause to enhance security and improve safety
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants