Skip to content

Fix IDOR on attendee selection response#3150

Merged
brage-andreas merged 3 commits into
dotkom:mainfrom
ch0rizo:fix-attendee-selection-idor
Apr 30, 2026
Merged

Fix IDOR on attendee selection response#3150
brage-andreas merged 3 commits into
dotkom:mainfrom
ch0rizo:fix-attendee-selection-idor

Conversation

@ch0rizo
Copy link
Copy Markdown
Contributor

@ch0rizo ch0rizo commented Apr 29, 2026

Kjære dotkom,

Takk for at dere viste oss hvordan responsible disclosure gjøres🙏 Vi tok det til oss.

Vi fant at updateSelectionResponsesProcedure kun sjekket at du var innlogget - ikke at du faktisk eier attendee-recordet du prøver å endre. To linjer fikset det. Lar hvem som helst melde seg på pils og sprit på andres vegne🍺

Men siden vi først var innom kodebasen ofret vi et par tusen tokens til... og fant vi et par ting👀

  • user.findByUsername og user.getByUsername er helt åpne og returnerer full UserSchema - epost, telefon, kosthold og alt. Ingen auth. Ingen projeksjon. Bare ta deg til rette.
  • event.get, event.find og event.attendance.getAttendance er heller ikke autentisert, og embedder full UserSchema på alle attendees. Historisk masseeksponering av PII for alle som har meldt seg på et arrangement.

Dere har til og med laget PublicUserSchema - så dere er på god vei😇

Ingen interessegrupper for pils og sprit ble stiftet i prosessen.

Mvh Webkom❤️
PS: Vi er også den beste kodekomiteen på NTNU

@brage-andreas
Copy link
Copy Markdown
Member

goated

@brage-andreas brage-andreas changed the title fix: IDOR on attendee selection response Fix IDOR on attendee selection response Apr 30, 2026
Copy link
Copy Markdown
Member

@brage-andreas brage-andreas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@brage-andreas brage-andreas merged commit 3849a12 into dotkom:main Apr 30, 2026
1 of 4 checks 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.

2 participants