Panic message when query a IBC denom supply

[RaulBernal]

Summary of Bug

When you ask for a IBC denom supply you get a panic message:

{
  "code": 2,
  "message": "panic message redacted to hide potentially sensitive system info: panic",
  "details": [
  ]
}

https://cosmoshub.stakesystems.io/cosmos/bank/v1beta1/supply/0030B0AA8A1B9028703B1D17B965FBCBE0136571B854410BFEC3475B1DC765A1
https://cosmoshub.stakesystems.io/cosmos/bank/v1beta1/supply/977C824A11AF322AADA025C196ADC3A7F7409E342C35826106F9BF423AF32161

All the denoms with their amounts are here:
https://cosmoshub.stakesystems.io/cosmos/bank/v1beta1/supply

Version

some SDK v.0.44.x chains are affected

Steps to Reproduce

Ask a denom's supply:
https://cosmoshub.stakesystems.io/cosmos/bank/v1beta1/supply/0030B0AA8A1B9028703B1D17B965FBCBE0136571B854410BFEC3475B1DC765A1

---

For Admin Use

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

[alexanderbez]

Well the panic isn't good for sure, but I'm highly doubtful that's the correct URL. Try with ?denom=ibc/....

[tac0turtle]

@RaulBernal did you have a chance to test this?

[tac0turtle]

This seems like a bug, @alexanderbez ?denom=ibc/.. isn't supported here.

  rpc SupplyOf(QuerySupplyOfRequest) returns (QuerySupplyOfResponse) {
    option (google.api.http).get = "/cosmos/bank/v1beta1/supply/{denom}";
  }

[RaulBernal]

@RaulBernal did you have a chance to test this?

I didn't make it works

[amaury1093]

In #10394, for querying balances we changed the REST endpoint:

- "/cosmos/bank/v1beta1/balances/{address}/{denom}"
+ "/cosmos/bank/v1beta1/balances/{address}/by_denom?denom={ibc_denom}"

We should do the same here for supply

- "/cosmos/bank/v1beta1/supply/{denom}"
+ "/cosmos/bank/v1beta1/supply/by_denom?denom={ibc_denom}"

It's a client-breaking change, but IMO it's worth it because it fixes querying IBC denoms.