Skip to content

[codex] Harden authentication and member operations - #3

Merged
SH-ZMD merged 3 commits into
masterfrom
codex/backend-safety-fixes
Aug 21, 2026
Merged

SH-ZMD merged 3 commits into
masterfrom
codex/backend-safety-fixes

Conversation

@SH-ZMD

@SH-ZMD SH-ZMD commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Add bounded in-memory login and member-submission rate limiters.
  • Use Tomcat-native forwarded-header handling and the validated remote address for login limits.
  • Add the protected account-list endpoint needed by the existing admin account workflow.
  • Prevent administrators from disabling themselves or removing their own administrator role.
  • Limit audit-log reads to the newest 200 entries and return a consistent 415 response for unsupported content types.
  • Add focused limiter, controller, security, and account-service regression tests.

Why

The existing backend needed protection against repeated login and submission attempts, and the frontend account-management workflow lacked a safe list endpoint. The forwarded-address handling also needed to resist spoofed prefix values behind a proxy.

Validation

  • .\\gradlew.bat --no-daemon clean test bootJar: passed on Temurin JDK 25.0.3.
  • Real Tomcat proxy-limit check returned 401 for the first five attempts, 429 for the sixth attempt from the same validated address, and 401 from a different validated address.
  • Local health and public-home APIs returned HTTP 200.
  • git diff --check: passed.

@SH-ZMD
SH-ZMD marked this pull request as ready for review August 21, 2026 09:47
@SH-ZMD
SH-ZMD merged commit 165fa81 into master Aug 21, 2026
1 check passed
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.

1 participant