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

ValidateSelfClient missing in in ConnOpen{Try,Ack} #162

Closed
5 tasks
plafer opened this issue Oct 11, 2022 · 0 comments · Fixed by #207
Closed
5 tasks

ValidateSelfClient missing in in ConnOpen{Try,Ack} #162

plafer opened this issue Oct 11, 2022 · 0 comments · Fixed by #207
Labels
A: bug Admin: something isn't working
Milestone
v0.22.0

Comments

@plafer
Copy link
Contributor

plafer commented Oct 11, 2022

We're missing the validation that the ClientState included in the msg is as we expect it to be.

See ibc-go

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@plafer plafer added the A: bug Admin: something isn't working label Oct 11, 2022
@plafer plafer mentioned this issue Oct 11, 2022
Merged
7 tasks
livelybug added a commit to octopus-network/ibc-rs that referenced this issue Oct 14, 2022
@livelybug
Merge pull request cosmos#162 from octopus-network/feature/remove-sleep
d29838f 
remove some sleep replace retry_with_index
@plafer plafer added this to the v0.21.0 milestone Oct 19, 2022
@plafer plafer mentioned this issue Oct 21, 2022
Closed
5 tasks
@plafer plafer mentioned this issue Oct 31, 2022
Merged
7 tasks
shuoer86 pushed a commit to shuoer86/ibc-rs that referenced this issue Nov 4, 2023
@faddat @crodriguezvega
chore: upgrade go (cosmos#162)
9a555c4 
* upgrade golang in ics23

* update ci config

---------

Co-authored-by: Carlos Rodriguez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: bug Admin: something isn't working
Projects
None yet
Milestone
v0.22.0
Development

Successfully merging a pull request may close this issue.

Add validate_self_client() cosmos/ibc-rs
1 participant
@plafer