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

ValidatorInfo rpc should return all validators, not just active and inactive ones #4723

Open
Valentine1898 opened this issue Jul 18, 2024 · 0 comments
Labels
A-client Area: Design and implementation for client functionality _P-high High priority _P-V2 Priority: after mainnet

Comments

@Valentine1898
Copy link
Contributor

Is your feature request related to a problem? Please describe.

ValidatorInfo rpc returns data only for active and inactive validators. This means clients cannot discover its existence if a validator is jailed or tombstoned.
This is especially critical for prax, which during modern epoch sync must retrieve all validators and store them in indexed-db to be able to display the user's stakes

By syncing only the modern block, we may lose some validators and stakes.
Example:
Epoch 2: the validator has joined the network
Epoch 3: user delegates tokens to validator
Epoch 4: validator jailed

If the user syncs from the beginning after epoch 5 and the validator remains jailed, we will never know about the existence of this validator and will lose the user stake

Also, it's an issue for other clients too. I think that for example, https://penumbra.today will not be able to display jailed validators

Describe the solution you'd like
ValidatorInfo rpc should return data of all validators regardless of their status

Additional context
related to penumbra-zone/web#1453

@github-actions github-actions bot added the needs-refinement unclear, incomplete, or stub issue that needs work label Jul 18, 2024
@Valentine1898 Valentine1898 changed the title ValidatorInfo rpc should return data from all validators, not just active and inactive ones ValidatorInfo rpc should return all validators, not just active and inactive ones Jul 18, 2024
@aubrika aubrika added _P-V2 Priority: after mainnet _P-high High priority A-client Area: Design and implementation for client functionality and removed needs-refinement unclear, incomplete, or stub issue that needs work labels Aug 7, 2024
@aubrika aubrika added this to the Sprint 10 milestone Aug 12, 2024
@aubrika aubrika removed this from the Sprint 10 milestone Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-client Area: Design and implementation for client functionality _P-high High priority _P-V2 Priority: after mainnet
Projects
Status: Backlog
Development

No branches or pull requests

2 participants