Skip to content

Ignore IDOR level 4 encoded role cookie - #230

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

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

Conversation

@r1kka6

@r1kka6 r1kka6 commented Aug 10, 2026

Copy link
Copy Markdown

Summary

  • stop trusting the Base64-encoded role_level4 cookie for authorization
  • derive the effective role from the validated token claims only
  • update Level 4 unit coverage to reject encoded admin escalation attempts

Verification

  • ./gradlew test spotlessJavaCheck bootJar --no-daemon passed in an eclipse-temurin:17-jdk container
  • vulnerable baseline lets Alice read Bob by changing role_level4 to Base64 ADMIN (QURNSU4)
  • patched runtime keeps Alice's own profile as USER and rejects id=2 even when role_level4=QURNSU4 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