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

Add a method to the Validator Manager so as to obtain the list of all the validators in the subnet #594

Open
felipemadero opened this issue Oct 12, 2024 · 2 comments
Labels

Comments

@felipemadero
Copy link
Collaborator

Context and scope
There is no way to obtain such a list from the contract itself.

Discussion and alternatives
Include a description of the changes to be made to the code along with alternatives
that were considered, including pro/con analysis where relevant.

Open questions
Questions that are still being discussed.

@cam-schultz
Copy link
Contributor

Is there any reason why GetValidatorsAt can't be used instead? This is what we use in our E2E tests to construct Warp signatures.

Exposing this in the contract would require us to maintain additional state in the contract, since there's no way to extract all set keys from a mapping in Solidity.

@felipemadero
Copy link
Collaborator Author

Is there any reason why GetValidatorsAt can't be used instead? This is what we use in our E2E tests to construct Warp signatures.

Exposing this in the contract would require us to maintain additional state in the contract, since there's no way to extract all set keys from a mapping in Solidity.

I see. I thought that was an option in Solidity. I believe the contract will be much more self contained providing this
ofc important information. We at CLI can manage to use P-Chain.

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

No branches or pull requests

3 participants