OAuth2 consent page: unbounded scope list can push the approval buttons off-screen
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- Meio dia
- Facilidade para iniciantes
- 68/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Ativa
- Stack de tecnologia
- css, html
- Domínio
- frontend
Direção de pesquisa
Abra site/static/oauth2allow.html e inspecione #scope-list, body e .container. Renderize a página de consentimento com comprimentos crescentes de scope-list em uma altura mínima de janela realista para identificar quando os botões saem da viewport. Considera-se concluído quando todos os scopes negociados permanecem visíveis e os botões Allow e Cancel continuam acessíveis por rolagem sem truncar a lista.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Raised as CRF-10 in the round-3 review of #28179, deliberately deferred rather than fixed there.
site/static/oauth2allow.html renders the negotiated scope as #scope-list, which has no upper bound on length. body flex-centers .container inside a viewport-height parent with align-items: center, and .container has no overflow-y: auto. A grant with enough scope names could therefore push the Allow and Cancel buttons below the fold, on the one page where the user has to be able to reach them.
Unverified by rendering. The reviewer reasoned from the stylesheet rather than from a screenshot, so the first step is to establish at what list length (if any) the buttons actually leave the viewport, at a realistic minimum window height.
If it does reproduce, the fix is a scroll container on the list or on .container rather than a cap on the number of scopes shown: truncating the list on a consent page would hide permissions the user is being asked to approve, which is worse than scrolling.
Related: #28179, and CRF-3 (scope names carry no user-facing description) which touches the same block.
Refs PLAT-479.
- Linguagem predominante
- Go
- Estrelas
- 16.6k
- Forks
- 1.6k
- Merge médio
- 1d 23h
- PRs com merge (30d)
- 543
Guia de contribuição
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 coder/coder
-
bug: AI Gateway client filter lists "Unknown" twice when NULL and literal Unknown clients coexist Abertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 90/100
-
feat(site): suppress the web terminal context menu when the application has enabled mouse tracking Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
-
bug frontend
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
-
bug site
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
Todas as issues de coder/coder
Issues semelhantes
-
agentic-workflows
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
-
agentic-workflows
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
microsoft/agent-framework-go#1179 ·
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
-
[Bug]: OLLAMA_KEEP_ALIVE="5m" / "24h" crashes Ollama embedding and vision models with ValueError Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
infiniflow/ragflow#20223 · 1 reação ·
-
bug needs triage pkg/translator/faro
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
open-telemetry/opentelemetry-collector-contrib#51484 · 1 comentário ·