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

Clarify validator id as any bytes48 #184

Merged
merged 1 commit into from
Jan 11, 2022
Merged

Clarify validator id as any bytes48 #184

merged 1 commit into from
Jan 11, 2022

Commits on Jan 10, 2022

  1. Clarify validator id as any bytes48 

    Validator ID in a query parameter should not be required to be on the g2 curve to be valid, any bytes48 should be allowed to be queried on.

Reading in any bytes48 without validating against the g2 curve will reduce overheads of parsing large lists, and given that valid missing keys are just not returned, there are no real negatives to querying a key that will definitely be missing.

fixes ethereum#183
    @rolfyone
    rolfyone committed Jan 10, 2022
    Configuration menu
    Copy the full SHA
    dff3e50 View commit details
    Browse the repository at this point in the history