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

add warnings for invalid el triggered consolidations & withdrawals #238

Merged

pk910 merged 2 commits into master from pk910/warnings-for-invalid-operations

Feb 17, 2025

Member

pk910 commented Feb 9, 2025 •

edited

Loading

adds warnings for probably invalid requests:

there are warnings for:

consolidation requests to a validator without 0x02 credentials
consolidation request from a validator that is younger than the shard committee period (not consolidate-able)
withdrawal request for a validator that is younger than the shard committee period (not withdrawable)

fixes #229, but keeps the possibility to submit invalid requests for testing purposes.
cc @remyroy


          add warnings for invalid el triggered consolidations & withdrawals

bc150c6

Member Author

pk910 commented Feb 9, 2025

#236 should also make the outcome of requests more clear.


          Merge branch 'master' into pk910/warnings-for-invalid-operations

9f14bec

skylenet approved these changes

View reviewed changes

pk910 merged commit 562267b into master

4 checks passed

pk910 deleted the pk910/warnings-for-invalid-operations branch

February 17, 2025 15:36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet