ICS PoC

Missing specs

• Router
• Handler

What we don't understand in the spec

• In the connection spec, why the datagrams should have ConsensusState proof?
• When the recv packet does not need acknowledgement.

We need these things in the final implementation

• Module system

• Creating a connection needs a vote system

  Make only authorized users be able to create a connection · Issue #159 · CodeChain-io/foundry

ICS needs event logging system

Different things from Cosmos implementation

• Foundry's light client needs every blocks. Cosmos' light client can jump blocks with losing some security.

Design for final implementation

• light client module interface

  Remove double hashing in state db access · Issue #167 · CodeChain-io/foundry

Strange things in spec

• In the channel spec, no port & channel in log: emitLogEntry("sendPacket", {sequence: packet.sequence, data: packet.data, timeout: packet.timeout})

Minor misses in the PoC implementation

• When saving IBC data, the key is hashed twice. https://github.com/CodeChain-io/foundry/issues/167
• We couldn't update light client upto the best block because there is no easy way to get precommits for the best block now.

Not specified in the spec yet:

• Version
• Connection Hops