Skip to content

feat(auth): limit challenge and signature abuse - #198

Merged
Heazzy500 merged 6 commits into
Local-Settle:mainfrom
mayborn005:fix/issue-45
Oct 18, 2026
Merged

Heazzy500 merged 6 commits into
Local-Settle:mainfrom
mayborn005:fix/issue-45

Conversation

@mayborn005

Copy link
Copy Markdown
Contributor

Summary

  • Add configurable fixed-window limits for challenge issuance by IP and wallet, plus a separate limit for failed signature attempts.
  • Record sanitized audit events for issued challenges and rate-limit rejections; hashed wallet identifiers are used in auth audit records.
  • Document thresholds and legitimate retry behavior, with focused regression coverage.

Closes #45

Checks

  • git diff --check passed.
  • Jest could not run because dependencies are not installed (jest: command not found); no install or build was run.

@drips-wave

drips-wave Bot commented Oct 17, 2026

Copy link
Copy Markdown

@mayborn005 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Heazzy500
Heazzy500 merged commit 01d82ed into Local-Settle:main Oct 18, 2026
1 of 5 checks passed
@mayborn005

Copy link
Copy Markdown
Contributor Author

@Adesokan-2 I resolved the merge conflict and updated the branch. The remaining lint failure is an unused import already present on the current base branch; could you review the PR and advise on that shared check?

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.

Add login challenge attempt limits by wallet and IP

2 participants