Skip to content

Remove password hashes from legacy user API responses - #288

Open
r1kka6 wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
r1kka6:fix/api-security-low
Open

r1kka6 wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
r1kka6:fix/api-security-low

Conversation

@r1kka6

@r1kka6 r1kka6 commented Aug 9, 2026

Copy link
Copy Markdown

Summary

  • remove the password field from legacy v1 user serialization
  • keep v1 and v2 user responses aligned so changing API versions no longer exposes password hashes

Verification

  • docker run --rm -v "$PWD":/work -w /work php:8.3-cli php -l vulnerabilities/api/src/User.php
  • vulnerable baseline returns password hashes from /vulnerabilities/api/v1/user/
  • patched runtime returns only id, name, and level from both v1 and v2 user endpoints

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