cosmos-sdk: error while using map in txs

Summary of Bug

I am trying to use map in a transaction message. And the txn fails with code 2 and error message does not implement proto.Message.

Version

main

Steps to Reproduce

I have added a unit test to reproduce this error. Clone https://github.com/0xmuralik/cosmos-sdk Switch to branch murali/map-in-tx-msg And run the test.

About this issue

Original URL
State: closed
Created a year ago
Comments: 15 (15 by maintainers)

Commits related to this issue

Convert sub-objects (other than arrays) in YAML to JSON strings automatically. (#38) This allows us to support attributes in YAML like this: ``` meta: foo: bar bar: baz: boz ``` Whi... — committed to cerc-io/laconic-registry-cli by telackey 7 months ago

Most upvoted comments

We are using maps to reference an IPLD CID. Each field will have only one element in the map. Hence, this will not lead to non-determinisim. Example here.

0xmuralik on Mar 6, 2023

still lost on the error, id like to dive into it later this week or next and make sure we have documentation around such things. Ill keep the issue open and assign myself

tac0turtle on Mar 8, 2023

@alexanderbez to your comment, we are trying to match standards used elsewhere in IPLD https://github.com/ipld/specs/blob/master/block-layer/codecs/dag-json.md#format

i-norden on Mar 8, 2023

In any case, this issue falls well outside the concerns/responsibilities of the SDK. We can make work arounds on our end. Thanks! @0xmuralik @alexanderbez @tac0turtle we can close this.

i-norden on Mar 8, 2023

You cannot use maps, which are non-deterministic, in any state machine proto type.

alexanderbez on Mar 5, 2023