Christopher Goes

Results 299 comments of Christopher Goes

The `store` and `cdc` are Cosmos SDK internal state-accessor / state-decoder interfaces, I don't think they should be included in the IBC specifications. The IBC specification generally uses `get`, `set`,...

Right, I'll clarify those bits. > On a different note... I might be missing something but it looks like the client verification functions can be generic. They just seem to...

> I was browsing a bit the SDK client code and it is hard to see what is really client specific. For example the verify functions take a prefix and...

... this means we need to keep state on a past range of public keys

In principle this makes sense to me. How exactly would the `CLOSED` verification work, though, considering that the handshake failed? I think we need to be a little careful lest...

Hi @bluele - thanks for the PR! We will try to take a look at it soon.

In general, at the moment, it is safe to prune connections not in use by any channel, but unsafe to prune channels once they have been opened, since there's no...

I agree with this generally; I think we should consider it alongside the wider restructure outlined in https://github.com/cosmos/ics/issues/468.

Regarding proposals and genesis, see my comment here - https://github.com/cosmos/ics/issues/497#issuecomment-732541353. Regarding upgrades, I am also not convinced this should be in the ICS specs - it is really a hack...

> Thus, for a chain that does not have absence proofs, a relayer need only attempt a `RecvPacket` on the receiver chain to fill in the absence receipt on the...