Technical Debt Tracker

[AdityaSripal]

This issue is a tracker for technical debt accumulated after IBC 1.0 for the purpose of maintaining backward compatibility.

• Indirect Iteration Keys needed for efficient iteration while not breaking ICS-24 paths which are inefficient for iteration: Efficient Consensus State Iteration #125
• Maintaining FrozenHeight in clientState even though it will not be used except as a boolean (non-zero vs zero height): Time Monotonicity Enforcement #141
• Maintaining DEPRECATED packet_data attribute, users should switch to packet_data_hex: Hex Packet Data #144
• IBC client/consensus states for upgrades are set in the Upgrade Keeper
• v0.42.0 SDK to ibc-go v1.0.0 migration code ie legacy/v100 ref: Add in-place and genesis migrations #205
• solo machine bypass for timestamp check disable defensive timestamp check for solo machines in SendPacket #214

[colin-axner]

tendermint#6403 may influence which attribute should be deprecated between packet_data and packet_data_hex. We can discuss with relayers which is preferred when the time comes

[crodriguezvega]

All the PRs mentioned above haven been merged and I believe the upgrade keeper is already used to set client/consensus states. Thus I will close the issue now.

[colin-axner]

I believe the upgrade keeper is already used to set client/consensus states.

This is correct. The original issue is a bit hard to follow, but the intention was to note technical debt, so the fact that the client/consensus states being set by the upgrade keeper is actually a problem. Ideally they would exist under the client keeper.

I think a couple of the other issues mentioned are not addressed:

• frozen height is still maintained in the client state (it should be a boolean)
• duplicate event emission for packet_data is still maintained (should be removed)
• Indirect Iteration Keys needed for efficient iteration while not breaking ICS-24 paths which are inefficient for iteration (we should upgrade connections to allow for efficient iteration of consensus keys, thus making the indirect iteration keys useless)

[colin-axner]

I think we should actually move this issue to a document. I do think it is useful to document areas of the code which have been deprecated and should be fixed/removed when possible