Skip to content

Ignore IDOR level 3 role cookie - #229

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

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

Conversation

@r1kka6

@r1kka6 r1kka6 commented Aug 10, 2026

Copy link
Copy Markdown

Summary

  • stop trusting the client-controlled role_level3 cookie for authorization
  • derive the effective role from the validated token claims only
  • update the Level 3 unit test to reject cookie-based admin escalation

Verification

  • ./gradlew test spotlessJavaCheck bootJar --no-daemon passed in an eclipse-temurin:17-jdk container
  • vulnerable baseline lets Alice read Bob by changing role_level3=USER to role_level3=ADMIN
  • patched runtime keeps Alice's own profile as USER and rejects id=2 even when role_level3=ADMIN is supplied

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