Broken Trino impersonation
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- typescript
- Domaine
- authentication, authorization, backend
Piste de recherche
Start with the input: false setting on user.additionalFields.username, and trace why parseAdditionalUserInputFromProviderProfile drops the username that mapProfileToUser returns during OAuth user creation. Then check the Trino API routes that fall back to locals.user?.username ?? 'anonymous'. Done means the username survives OAuth signup, impersonation works again, the update-user restriction from #330 still holds, and regression tests cover both cases.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In #330 I set input: false on user.additionalFields.username to stop users from changing it through /api/auth/update-user.
But better-auth also applies input: false to the OAuth provider profile: parseAdditionalUserInputFromProviderProfile skips such fields, so the username returned by mapProfileToUser is dropped when the user is created.
Without a username, the Trino API routes fall back to locals.user?.username ?? 'anonymous', which hides the error and breaks impersonation.
- Fix and restore old behavior
- While also ensuring that the Issue in #330 can not be exploited
- Add regression tests
- Langage dominant
- TypeScript
- Étoiles
- 1
- Forks
- 0
- Merge moyen
- 3 j 23 h
- PR mergées (30 j)
- 15
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de stackabletech/cockpit
-
Remove unused dependenciesPeut-être pris @Bohreromir l’a pris il y a 23 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
stackabletech/cockpit#101 ·
Les mainteneurs répondent en général sous 1 jour
-
Parallelize testsOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
stackabletech/cockpit#381 ·
Les mainteneurs répondent en général sous 1 jour
-
localStorage InterfaceOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 22/100
stackabletech/cockpit#379 ·
Les mainteneurs répondent en général sous 1 jour
-
DocumentationOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
stackabletech/cockpit#378 ·
Les mainteneurs répondent en général sous 1 jour
-
Add DB vars to helm chartOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
stackabletech/cockpit#377 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de stackabletech/cockpit
Issues similaires
-
enhancement good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
OWASP/cve-lite-cli#1335 ·
Les mainteneurs répondent en général sous 1 jour
-
Windows: main-screen mouse tracking requires WT_SESSION, so it never enables inside herdr panesOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 66/100
code-yeongyu/senpi#3353 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Channels status: needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
mastra-ai/mastra#27093 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement needs-triage ui
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
Les mainteneurs répondent en général sous 1 jour