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

REST VC: Use POST to fetch validators #13239

Merged
merged 4 commits into from
Nov 29, 2023
Merged

REST VC: Use POST to fetch validators #13239

merged 4 commits into from
Nov 29, 2023

Conversation

rkapka
Copy link
Contributor

@rkapka rkapka commented Nov 29, 2023

What type of PR is this?

Feature

What does this PR do? Why is it needed?

Our REST validator client on Prater cannot start validating because of this error: failed to get state validators: HTTP request unsuccessful (431: Request Header Fields Too Large). This is happening because we issue a GET request to GetValidators with 1024 validator pubkeys in the query string, which is too long for the HTTP server. To mitigate issues such as this, a POST version of the GetValidators endpoint was proposed in the Beacon API spec: ethereum/beacon-APIs#367. This PR switches how the REST validator client obtains validators - it will use POST from now on.

@rkapka rkapka added Ready For Review A pull request ready for code review API Api related tasks Validator Client labels Nov 29, 2023
@rkapka rkapka requested a review from a team as a code owner November 29, 2023 15:06
@rkapka rkapka merged commit 5ecb4d6 into develop Nov 29, 2023
17 checks passed
@rkapka rkapka deleted the rest-vc-vals-post branch November 29, 2023 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Api related tasks Ready For Review A pull request ready for code review Validator Client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants