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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Waiting for Review

Development

Successfully merging this pull request may close these issues.

1 participant