ChainId parsing fails if revision number is absent

[rnbguy]

Bug Summary

ChainId::from_str("basecoin") fails as revision number is not given. ibc-go docs discusses that the clients without revision requirement should have hardcoded zero as revision number.

Details

Current ChainId::from_str uses parse_chain_id_string, which requires a revision number at the end of the string.

https://github.com/cosmos/ibc-rs/blob/553bc79b31b31c586128e4e533105fe917df4c0e/crates/ibc/src/core/ics24_host/identifier.rs#L139-L146

But ibc-go implementation doesn't require this and returns zero by default if the revision number is not specified.

Version

v0.47.0