Skip to content

Keep permanent ADNL validator keys#118

Merged
bvscd merged 2 commits into
release/node/v0.5.3from
fix_adnl_keys
Apr 23, 2026
Merged

Keep permanent ADNL validator keys#118
bvscd merged 2 commits into
release/node/v0.5.3from
fix_adnl_keys

Conversation

@bvscd
Copy link
Copy Markdown
Collaborator

@bvscd bvscd commented Apr 23, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 23, 2026 10:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts validator key rotation to retain ADNL validator keys that are still referenced by other elections, preventing premature cleanup of shared ADNL/QUIC/DHT state.

Changes:

  • Refactors revision_validator_keys control flow using early returns and increases the “no rotation” threshold to 3 keys.
  • Removes oldest validator key when exceeding the threshold, but conditionally skips ADNL key removal if still referenced.
  • Emits RemoveValidatorAdnlKey events only when the ADNL key is actually removed.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/node/src/config.rs
Comment thread src/node/src/config.rs
Comment thread src/node/src/config.rs Outdated
Comment thread src/node/src/config.rs
@bvscd bvscd merged commit 76c6c29 into release/node/v0.5.3 Apr 23, 2026
11 checks passed
@bvscd bvscd deleted the fix_adnl_keys branch April 23, 2026 11:47
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.

3 participants