Validating timeout failed when height timeout

[yito88]

Bug Summary

If timeout_height_on_b in a packet reached, the timeout handling failed.

Details

The handling returns PacketTimeoutHeightNotReached error if the timeout height has passed.

        if msg
            .packet
            .timeout_height_on_b
            .has_expired(msg.proof_height_on_b)
        {
            return Err(PacketError::PacketTimeoutHeightNotReached {
                timeout_height: msg.packet.timeout_height_on_b,
                chain_height: msg.proof_height_on_b,
            }
            .into());
        }

It should return an error only when both the timeout height and timestamp have not been reached.

Version

0.32.0