ClientState is stored in the privateStore in the standard, but is provable in ibc-go

[plafer]

ibc-go returns a proof in QueryClientStateResponse, suggesting that ClientState is stored in the provableStore, whereas the standard claims that ClientState is stored in privateStore (see ICS-24 and ICS-2).

The standard should probably be adjusted to remove this inconsistency.

[plafer]

Moreover, ibc-go relies on a proven ClientState for its ConnOpenTry() and ConnOpenAck() handlers, whereas the standard doesn't.

[AdityaSripal]

Indeed the clientstate must be stored in the public store.

Also, definitely need to add the client verification from ibc-go into the standard. Not sure why this was missed.

Thanks @plafer for finding these! Are you interested in submitting the fixes?

[plafer]

Yes sure I can do that!