Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove "crossing hellos" logic from connection handshake #156

Closed
Tracked by #143
plafer opened this issue Oct 7, 2022 · 0 comments · Fixed by #170
Closed
Tracked by #143

Remove "crossing hellos" logic from connection handshake #156

plafer opened this issue Oct 7, 2022 · 0 comments · Fixed by #170
Milestone

Comments

@plafer
Copy link
Contributor

plafer commented Oct 7, 2022

No description provided.

@plafer plafer added this to the v0.21.0 milestone Oct 19, 2022
shuoer86 pushed a commit to shuoer86/ibc-rs that referenced this issue Nov 4, 2023
Closes: informalsystems#156

Addresses the `no_std` compatibility issue with `serde` feature in `ibc-rs`. This is caused by the recent implementation of ProtoJSON serialization and deserialization [0] for the `ics23` Protobuf definitions using `pbjson`, and then re-exporting the ics23 type [1] in `ibc-proto-rs`. Some of our users by then (starting from IBC-rs v0.41.0) are experiencing compilation errors. [2]

To meet this immediate need [3] and the lack of activity in the `pbjson` crate for months, we have taken the initiative to feature `no_std` support in the `informalsystems-pbjson` crate and have it published.

[0] cosmos/ics23#146
[1] cosmos/ibc-proto-rs#92
[2] informalsystems#741
[3] cosmos/ibc-proto-rs#98 (comment)

---

* feat: enable no_std support for pbjson

* fix: get serde feature work with no-std

* deps: use informalsystems-pbjson v0.6.0

* deps: use informalsystems-pbjson v0.6.0
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant