Cosmos-sdk: Allow querying for genesis transactions on height 0

[faboweb]

Summary

It would be nice to be able to query the genesis block as height 0.

Problem Definition

We can query blocks, validatorsets, transactions and others by block height. For the block 0 (genesis block) we can't.

Proposal

It would be nice to unify this experience by indexing the block, transactions and others as height 0 even if they miss some properties.

---

For Admin Use

• [ ] Not duplicate issue
• [ ] Appropriate labels applied
• [ ] Appropriate contributors tagged
• [ ] Contributor assigned/self-assigned

[alexanderbez]

Why can't you get block 1?

[faboweb]

It's not the genesis block isn't it? So I wouldn't see the genesis transactions.

[fedekunze]

we can query the genesis txs using the /genesis rpc endpoint and parse the gen_txs property

[alexanderbez]

Great! Can we close this issue then?

[fedekunze]

we could do something like /txs/genesis or /txs?tx.height=0 to get the txs, bc technically it's not a block

[faboweb]

I think the biggest case is genesis transactions. So I change the title.

[jackzampolin]

@fedekunze why not make /txs?tx.height=0 parse genesis and pull out the gentx? (i.e. sugar)

[faboweb]

That's what he is suggesting I think ;)

[fedekunze]

@colin-axner want to take on this one when you have time?

The scope would be:

• pulling the gentxs from the genesis state and return them on a query if heigth=0

For the next iteration, we could add events for each of the genesis txs to make them queryable

[colin-axner]

@fedekunze yea I can take this on

[colin-axner]

What is the reasoning for not just adding a new endpoint as suggested above, /txs/genesis? I think this would be a cleaner implementation then using /txs?tx.height=0

[fedekunze]

👍 let’s go for that one instead

[faboweb]

I don't think /txs/genesis is a cleaner API. If I want to show all transactions I need to stitch again results of multiple requests together.
Also on a side note /txs/genesis conflicts with /txs/{TX_HASH}.

[colin-axner]

@faboweb lets do the cleaner API then, I think I can make /txs?tx.height=0 work

[faboweb]

Great!

[colin-axner]

Pausing work on this until #4751 is resolved and genutils is updated