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.

Sign up for GitHub

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

Jump to bottom

Provide StateRoot as Parameter to ListBeaconCommittees RPC #3991

Closed
rauljordan opened this issue Nov 13, 2019 · 2 comments
Closed

Provide StateRoot as Parameter to ListBeaconCommittees RPC #3991

rauljordan opened this issue Nov 13, 2019 · 2 comments
Labels
Enhancement New feature or request Priority: Medium Medium priority item

Comments

@rauljordan
Copy link
Contributor

Discussed in #3977, there are cases in which there could be a forked chain and committees might be different from the point of view of a node. If a state root can be passed in as a request parameter, the ListBeaconCommittees function can verify it and the client can have peace of mind the committees being returned are from the same chain it is requesting. This would be an enhancement on top of #3977,
@rauljordan rauljordan added Enhancement New feature or request Priority: Medium Medium priority item labels Nov 13, 2019
@rauljordan
Copy link
Contributor Author

What should the behavior be if the state root does not match @shayzluf @nisdas, should we be returning an error or some sort of response that mentions the chain is forked?

@rauljordan
Copy link
Contributor Author

Closing in favor of #4450

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement New feature or request Priority: Medium Medium priority item
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rauljordan