/block endpoint could be cleaner

[nckrtl]

Summary

I am playing around with the block endpoint that's available on port 26657, and I noticed some redundant header information. Im posting this as a feature request as I'm not sure if its really a bug.

Problem Definition

When you request for example the api endpoint :26657/block?height=10000, you see a section with block_meta and block. However, they both contain a header part that contain exactly the same information and is therefor being redundant.

Proposal

Remove the header part from block_meta or remove the header part from block. Perhaps choosing the first option is best as the header part is actually containing meta information about the block.

If there is a good reason why we need two headers with the same information, I'd like to know.

Thanks.

[faboweb]

+1

[fedekunze]

Agree. I think this issue should go on https://github.com/tendermint/tendermint instead

[faboweb]

Maybe, but imo the SDK API doesn't need to reflect the Tendermint API.

[fedekunze]

Moved to tendermint/tendermint#2689