Philippe Laferrière
Philippe Laferrière
ICS-2 [treats `Height` as an opaque type](https://github.com/cosmos/ibc/blob/master/spec/core/ics-002-client-semantics/README.md#height) defined by a client type, while [ICS-7 describes the `Height`](https://github.com/cosmos/ibc/blob/master/spec/client/ics-007-tendermint-client/README.md#height) as we currently know it. We currently assume that all clients will use...
Preemptively approved, thanks a lot for going through this @plafer! It's a bit unfortunate that we now cannot statically rely on the proof being present in the returned type when...
The proof verification code, especially related to `ClientState` verification, is currently confusing and incomplete. The following 3 points should make the code better: 1. Use more meaningful variable names. +...
## Summary of Bug `conn_open_try::process()` and `conn_open_ack::process()` both use `verify_proofs()`. However, `verify_proofs()` succeeds the proof verification when the message contains no client_state (see [here](https://github.com/informalsystems/ibc-rs/blob/c3bdd507765334b34b6b8339e382ebf7b6660feb/modules/src/core/ics03_connection/handler/verify.rs#L32)), whereas ibc-go fails the verification (see...
## Summary of Bug With the refactor of #2226, I realized that the without proof didn't specify a height, whereas the query with proof (formerly in `proven_packet()`) did. gRPC queries...
## Summary of Bug Looking at `ChainEndpoint::proven_packet(PacketMsgType::TimeoutOnClose)` for `CosmosSdkChain`, we build a `ReceiptPath`, which is only correct if the channel is unordered. For ordered channels, we should query the `SeqRecvsPath`...
Is there a reason why `ClientState.verifyClientMessage()` throws an exception as opposed to returning a boolean? I would have expected it to return a boolean and be called inside a `abortTransactionUnless()`...
Branch: `testnet` I'm trying to build a [`StructTag`](https://github.com/MystenLabs/sui/blob/133fd85bcd79ed048a897be3a3f3bd4a2e9c616a/external-crates/move/move-core/types/src/language_storage.rs#L117), however I have no way to import the type, since all sui crates use the unpublished `move-core-types` v0.0.4 crates. Publishing it would...
The current [`Page`](https://mystenlabs.github.io/sui/sui_json_rpc_types/struct.Page.html) API is limiting for 2 reasons: 1. There is no way to parallelize the fetch. 2. There is no way to know how many pages there are....
The [section](https://github.com/cosmos/ibc/tree/main/spec/core/ics-004-channel-and-packet-semantics#acknowledgement-envelope) states: >The field numbers 21 and 22 were explicitly chosen to avoid accidental conflicts with other protobuf message formats used for acknowledgements. The first byte of any message...