Skip to content

feat: single-node implementation of slash-protection signer (backport #20894)#21158

Closed
AztecBot wants to merge 1 commit intobackport-to-v4-stagingfrom
claudebox/backport-slash-protection-signer-v4
Closed

feat: single-node implementation of slash-protection signer (backport #20894)#21158
AztecBot wants to merge 1 commit intobackport-to-v4-stagingfrom
claudebox/backport-slash-protection-signer-v4

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 5, 2026

Backport of #20894 to v4.

The automatic cherry-pick failed due to conflicts because on v4 the HA signing types had already been moved from stdlib/src/ha-signing/ to validator-ha-signer/.

Key adaptations for v4

  • Replaced @aztec/stdlib/ha-signing imports with local imports within validator-ha-signer
  • Added HA_SIGNER_* metric definitions and HA_DUTY_TYPE/HA_NODE_ID attributes to telemetry-client
  • Updated SlashingProtectionService and ValidatorHASigner constructors to accept deps parameter (metrics + dateProvider)
  • Added new package exports: ./local-config, ./metrics
  • Added LMDB-backed local signing protection database (LmdbSlashingProtectionDatabase)
  • Added createLocalSignerWithProtection factory function for single-node setups
  • Updated all test files to use new constructor signatures

ClaudeBox log: http://ci.aztec-labs.com/0cd974dff4a32c53-1

…20894)

Cherry-pick of c5cc991 with conflict resolution for v4.

Key adaptations for v4:
- Replaced @aztec/stdlib/ha-signing imports with local imports within
  validator-ha-signer (types already moved to this package on v4)
- Added HA_SIGNER_* metric definitions and HA_DUTY_TYPE/HA_NODE_ID
  attributes to telemetry-client (these were added in a prior PR on next)
- Updated SlashingProtectionService and ValidatorHASigner constructors
  to accept deps parameter (metrics + dateProvider)
- Added new package exports: ./local-config, ./metrics
- Added LMDB-backed local signing protection database
- Updated all test files to use new constructor signatures
@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 5, 2026
@spypsy
Copy link
Member

spypsy commented Mar 5, 2026

not needed for v4

@spypsy spypsy closed this Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants