polymerdao / ibc-go

Interblockchain Communication Protocol (IBC) implementation in Golang.
https://ibc.cosmos.network/
MIT License
11 stars 13 forks source link

Improve Endpoint interface abstraction #15

Closed alfredo-stonk closed 10 months ago

alfredo-stonk commented 1 year ago

Endpoint interface for multihop proof generation

Remove

Motivation: One client's ConsensusState height is normally the latest height of the clientState. But in cases of a local IBC connection (eg. Polymer's virtual IBC connections), this is no longer the case. GetClientConsensusHeight abstract away the actual logic of consensus height retrieval from the proof generation process. This change makes proof gen more robust for both remote and local ibc connections.

dshiell commented 1 year ago

I incorporated the non-conflicting changes from this PR after discussion with @7AC - I think we can close this out now.