Skip to content

Conversation

@jakubzadroga
Copy link
Contributor

@jakubzadroga jakubzadroga commented Nov 13, 2025

Added optional RPC authentication rate-limiting to protect nodes from brute-force or dictionary attacks. When enabled, repeated failed logins trigger an exponential backoff, temporarily blocking further attempts. All parameters — base delay, maximum delay, and attempt expiry — are fully configurable programmatically and via node.conf, while the default behaviour remains unchanged (no rate limiting unless explicitly enabled).

PR Checklist:

  • Have you run the unit, integration and smoke tests as described here?
  • If you added public APIs, did you write the JavaDocs/kdocs?
  • If the changes are of interest to application developers, have you added them to the changelog, and potentially the release notes (https://docs.r3.com/release-notes.html)?
  • If you are contributing for the first time, please read the contributor agreement now and add a comment to this pull request stating that your PR is in accordance with the Developer's Certificate of Origin.

Thanks for your code, it's appreciated! :)

@jakubzadroga jakubzadroga marked this pull request as draft November 13, 2025 16:09
@jakubzadroga jakubzadroga changed the title ENT-14229: Limit RPC login attempts ENT-14161: Limit RPC login attempts Nov 13, 2025
@jakubzadroga jakubzadroga marked this pull request as ready for review November 17, 2025 12:20
@jakubzadroga jakubzadroga requested a review from rick-r3 December 18, 2025 11:54
Copy link
Contributor

@rick-r3 rick-r3 left a comment

Choose a reason for hiding this comment

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

I think the algorithm and implementation is now basically right. Just made some detailed comments on the code now.

@jakubzadroga jakubzadroga requested a review from rick-r3 December 18, 2025 16:00
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.

4 participants