OAuth2 consent page: unbounded scope list can push the approval buttons off-screen
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- Mezza giornata
- Idoneità per principianti
- 68/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- css, html
- Ambito
- frontend
Direzione di ricerca
Apri site/static/oauth2allow.html e ispeziona #scope-list, body e .container. Esegui il rendering della pagina di consenso con lunghezze crescenti di scope-list a un’altezza minima realistica della finestra per identificare quando i pulsanti escono dal viewport. Il lavoro è completato quando tutti gli scope negoziati rimangono visibili e i pulsanti Allow e Cancel rimangono raggiungibili tramite scorrimento senza troncare l’elenco.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Go
- Stelle
- 16.6k
- Fork
- 1.6k
- Merge medio
- 1g 23h
- PR unite (30g)
- 543
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di coder/coder
-
bug: AI Gateway client filter lists "Unknown" twice when NULL and literal Unknown clients coexist Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
-
feat(site): suppress the web terminal context menu when the application has enabled mouse tracking Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
bug frontend
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
bug site
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Issue simili
-
agentic-workflows
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
agentic-workflows
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
microsoft/agent-framework-go#1179 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
[Bug]: OLLAMA_KEEP_ALIVE="5m" / "24h" crashes Ollama embedding and vision models with ValueError Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
infiniflow/ragflow#20223 · 1 reazione ·
-
bug needs triage pkg/translator/faro
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
open-telemetry/opentelemetry-collector-contrib#51484 · 1 commento ·