search

cosmos / ibc-go

Inter-Blockchain Communication Protocol (IBC) implementation in Golang.
https://ibc.cosmos.network/
MIT License
548 stars 586 forks source link

Refactor timeout handling to use timestamp only in channel/v2 handlers #7414

Open damiannolan opened 2 weeks ago

damiannolan commented 2 weeks ago

Summary

The on the feat/ibc-eureka branch we're currently using the existing v1 type for Timeout which requires it to be fulfilled with both height AND timestamp before calling timeout.Elapsed.

This timeout construction is no longer needed in v2 handlers as we will rely on timeout timestamps only. We can simply this code and not use the Timeout type, as well as removal any usage of heights in timeout handling.

This can be completed for all packet handlers in channel/v2.

For Admin Use

AdityaSripal commented 1 week ago

Also the timeout handling must use SECONDS instead of NANOSECONDS