Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: query redelegation cli command not working #21855

Closed
1 task done
akhilkumarpilli opened this issue Sep 23, 2024 · 2 comments · Fixed by #21856
Closed
1 task done

[Bug]: query redelegation cli command not working #21855

akhilkumarpilli opened this issue Sep 23, 2024 · 2 comments · Fixed by #21856
Assignees
Labels

Comments

@akhilkumarpilli
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

When trying to query redelegations using <app> query staking redelegation <delegator-addr> <src-validator-addr> <dst-validator-addr> command, got below error:

invalid argument "cosmosvaloper19nw35h4tu4ja7a2c3yryt9ewnr9gj9qr8csjve" for "--1" flag: invalid account address or key name: hrp does not match bech32 prefix: expected 'cosmos' got 'cosmosvaloper': internal logic error [cosmos/[email protected]/codec/address/bech32_codec.go:89]

Cosmos SDK Version

v0.50, v0.52, main

How to reproduce?

No response

@julienrbrt
Copy link
Member

Looks like the pulsar annotation are wrong in:

string dst_validator_addr = 3 [(cosmos_proto.scalar) = "cosmos.AddressString"];

In should have been cosmos.ValidatorAddressString when it is a validator.

@akhilkumarpilli
Copy link
Contributor Author

Looks like the pulsar annotation are wrong in:

cosmos-sdk/x/staking/proto/cosmos/staking/v1beta1/query.proto

Line 310 in f20e791

string dst_validator_addr = 3 [(cosmos_proto.scalar) = "cosmos.AddressString"];
In should have been cosmos.ValidatorAddressString when it is a validator.

Yes, raised a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🥳 Done
Development

Successfully merging a pull request may close this issue.

2 participants