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

feat: avs identifier callback and handle ejected operators #399

Open
wants to merge 5 commits into
base: test/slashing-release
Choose a base branch
from

Conversation

stevennevins
Copy link
Collaborator

@stevennevins stevennevins commented Feb 12, 2025

Prevents unauthorized cross-AVS registrations when multiple AVSs share an AVS Registrar.

Changes:

  • Add avs address to SlashingRegistryCoordinator interface
  • avs value will be used as a consent check to verify registrar assignment before an AVS can set it
  • Associated with the update in the core contracts that will utilize the getter

ejectOperator in SlashingRegistryCoordinator now only uses _forceDeregisterOperator, allowing AllocationManager to handle callbacks for deregistration

This change improves the ejection flow by ensuring proper deregistration handling through the AllocationManager while maintaining special handling for M2 quorums on ejection

stevennevins and others added 3 commits February 18, 2025 16:20
Changes:

- ejectOperator in SlashingRegistryCoordinator now only uses
_forceDeregisterOperator, allowing AllocationManager to handle callbacks
for deregistration

- RegistryCoordinator now handles M2 and non-M2 quorums separately:

- M2 quorums use _deregisterOperator

- Non-M2 quorums use _forceDeregisterOperator

- Added bitmap intersection support to filter quorum types properly

This change improves the ejection flow by ensuring proper deregistration
handling through the AllocationManager while maintaining special
handling for M2 quorums on ejection
@stevennevins stevennevins changed the title feat: avs identifier to be used in callback in src feat: avs identifier callback and handle ejected operators Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant