Skip to content

fix(auth): prevent users from gaining additional permissions - #330

Open
Bohreromir wants to merge 1 commit into
mainfrom
fix-trino-permission-escalation
Open

Bohreromir wants to merge 1 commit into
mainfrom
fix-trino-permission-escalation

Conversation

@Bohreromir

Copy link
Copy Markdown
Contributor

Users were able to pose as any other user in trino queries.
Can be done by logging in as bob and executing

await fetch('/api/auth/update-user', {
  method: 'POST',
  headers: { 'content-type': 'application/json' },
  credentials: 'include',
  body: JSON.stringify({ username: 'alice' })
});

in the browser console.
The fix prevents the username to be changed

@Bohreromir
Bohreromir marked this pull request as ready for review September 24, 2026 09:29
@Bohreromir Bohreromir moved this to Development: Waiting for Review in Stackable Engineering Sep 24, 2026
@dervoeti dervoeti moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Sep 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: In Review

Development

Successfully merging this pull request may close these issues.

2 participants