Skip to content

feat: add unsafe RPC to unban peers#666

Merged
gilescope merged 2 commits into
mainfrom
ozgb-unban-peer
Feb 13, 2026
Merged

feat: add unsafe RPC to unban peers#666
gilescope merged 2 commits into
mainfrom
ozgb-unban-peer

Conversation

@ozgb

@ozgb ozgb commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

Overview

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

Ticket: https://shielded.atlassian.net/browse/PM-21896

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).
@ozgb ozgb requested a review from a team as a code owner February 13, 2026 12:30
@github-actions

Copy link
Copy Markdown
Contributor

kics-logo

KICS version: v2.1.16

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 96
LOW LOW 12
INFO INFO 83
TRACE TRACE 0
TOTAL TOTAL 191
Metric Values
Files scanned placeholder 31
Files parsed placeholder 31
Files failed to scan placeholder 0
Total executed queries placeholder 73
Queries failed to execute placeholder 0
Execution time placeholder 9

@gilescope gilescope added this pull request to the merge queue Feb 13, 2026
Merged via the queue into main with commit 2f54f0d Feb 13, 2026
37 checks passed
@gilescope gilescope deleted the ozgb-unban-peer branch February 13, 2026 15:41
ozgb added a commit that referenced this pull request Feb 16, 2026
* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file
ozgb added a commit that referenced this pull request Feb 18, 2026
* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file
ozgb added a commit that referenced this pull request Feb 18, 2026
* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file
ozgb added a commit that referenced this pull request Feb 18, 2026
* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file
ozgb added a commit that referenced this pull request Feb 18, 2026
* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file
ozgb added a commit that referenced this pull request Feb 18, 2026
* chore: update indexer commit to fix sbom checks

* feat: add network_peerReputations and network_peerReputation RPC endpoints (#649)

* feat: add network_peerReputations and network_peerReputation RPC endpoints

Expose peer reputation and ban status via JSON-RPC to enable debugging
peer connectivity issues without custom tooling.

* docs: update change file

* chore: cargo fmt

---------

Co-authored-by: Squirrel <giles.cope@shielded.io>

* feat: add unsafe RPC to unban peers (#666)

* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file

---------

Co-authored-by: Squirrel <giles.cope@shielded.io>
ozgb added a commit that referenced this pull request Feb 18, 2026
* chore: update indexer commit to fix sbom checks

* feat: add network_peerReputations and network_peerReputation RPC endpoints (#649)

* feat: add network_peerReputations and network_peerReputation RPC endpoints

Expose peer reputation and ban status via JSON-RPC to enable debugging
peer connectivity issues without custom tooling.

* docs: update change file

* chore: cargo fmt

---------

Co-authored-by: Squirrel <giles.cope@shielded.io>

* feat: add unsafe RPC to unban peers (#666)

* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file

---------

Co-authored-by: Squirrel <giles.cope@shielded.io>
ozgb added a commit that referenced this pull request Feb 18, 2026
* chore: update indexer commit to fix sbom checks

* feat: add network_peerReputations and network_peerReputation RPC endpoints (#649)

* feat: add network_peerReputations and network_peerReputation RPC endpoints

Expose peer reputation and ban status via JSON-RPC to enable debugging
peer connectivity issues without custom tooling.

* docs: update change file

* chore: cargo fmt

---------

Co-authored-by: Squirrel <giles.cope@shielded.io>

* feat: add unsafe RPC to unban peers (#666)

* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file

* fix: remove non-determinism (#679)

* fix: keep existing preview ordering

Signed-off-by: Giles Cope <gilescope@gmail.com>

* preview has other tx with different orderings.

Just this change would not be enough to get to head.

* fix npm audit finding

---------

Signed-off-by: Giles Cope <gilescope@gmail.com>
Co-authored-by: Justin Frevert <justinfrevert@gmail.com>

* fix: sync non-deterministically while historical chain patch is pending (#685)

* fix: sync non-deterministically while historical chain patch is pending

* docs: add pr link to change file

* docs: add ticket link to change file

* chore: cargo fmt

---------

Signed-off-by: Giles Cope <gilescope@gmail.com>
Co-authored-by: Squirrel <giles.cope@shielded.io>
Co-authored-by: Justin Frevert <justinfrevert@gmail.com>
ozgb added a commit that referenced this pull request Feb 18, 2026
* fix: remove non-determinism (#679)

* fix: keep existing preview ordering

Signed-off-by: Giles Cope <gilescope@gmail.com>

* preview has other tx with different orderings.

Just this change would not be enough to get to head.

* fix npm audit finding

---------

Signed-off-by: Giles Cope <gilescope@gmail.com>
Co-authored-by: Justin Frevert <justinfrevert@gmail.com>

* fix: sync non-deterministically while historical chain patch is pending (#685)

* fix: sync non-deterministically while historical chain patch is pending

* docs: add pr link to change file

* docs: add ticket link to change file

* feat: add unsafe RPC to unban peers (#666)

* feat: add unsafe RPC to unban peers

Useful for node operators to use when a peer is banned but has since corrected it’s behaviour.

Without this RPC, the only way to reset peer reputation is to restart the node (reputation exists in memory only).

* docs: add pr link to change file

---------

Signed-off-by: Giles Cope <gilescope@gmail.com>
Co-authored-by: Squirrel <giles.cope@shielded.io>
Co-authored-by: Justin Frevert <justinfrevert@gmail.com>
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.

2 participants