Skip to content

Enforce IDOR level 1 profile ownership - #227

Open
r1kka6 wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
r1kka6:fix/idor-level-1
Open

r1kka6 wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
r1kka6:fix/idor-level-1

Conversation

@r1kka6

@r1kka6 r1kka6 commented Aug 10, 2026

Copy link
Copy Markdown

Summary

  • bind Level 1 profile reads to the user id from the validated token
  • reject cross-user id requests before querying the profile table
  • update unit coverage for denied cross-user access and allowed self access

Verification

  • ./gradlew test spotlessJavaCheck bootJar --no-daemon passed in an eclipse-temurin:17-jdk container
  • vulnerable baseline lets Alice read Bob with GET /IDORVulnerability/LEVEL_1?id=2
  • patched runtime keeps Alice's own id=1 response working and rejects id=2 with Access Denied - Insufficient privileges

Signed-off-by: snow <snow@snows-MacBook-Pro-2.local>
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