Broken Trino impersonation
Mantenedores costumam responder em até 2 dias
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Ativa
- Stack de tecnologia
- typescript
- Domínio
- authentication, authorization, backend
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 1
- Forks
- 0
- Merge médio
- 5d 16h
- PRs com merge (30d)
- 17
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de stackabletech/cockpit
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
stackabletech/cockpit#385 ·
Mantenedores costumam responder em até 2 dias
-
Remove unused dependenciesTalvez já em andamento @Bohreromir assumiu há 29 dias. Aberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 78/100
stackabletech/cockpit#101 ·
Mantenedores costumam responder em até 2 dias
-
Reduce number of e2e testAberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 45/100
stackabletech/cockpit#391 ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
stackabletech/cockpit#389 ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
stackabletech/cockpit#386 ·
Mantenedores costumam responder em até 2 dias
Todas as issues de stackabletech/cockpit
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
getsentry/sentry-conventions#732 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
status:in-linear status:team-assigned team:nodes
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
n8n-io/n8n#41548 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
motiondivision/motion#3955 ·
Mantenedores costumam responder em até 1 dia
-
DQ assisgnment closing at CRAbertaenhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
equinor/prisma-decision-web#919 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 83/100