Skip to content

Conversation

@stevennevins
Copy link
Contributor

@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
Copy link
Collaborator

@8sunyuan 8sunyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM now

@stevennevins stevennevins merged commit 25d3a94 into test/slashing-release Feb 20, 2025
5 checks passed
@stevennevins stevennevins deleted the feat/avs-identifier-src branch February 20, 2025 20:45
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.

5 participants