OAuth2 consent page: unbounded scope list can push the approval buttons off-screen
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 半日
- 初心者へのやさしさ
- 68/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- css, html
- 領域
- frontend
調査の方向性
site/static/oauth2allow.html を開き、#scope-list、body、.container を調べます。現実的な最小ウィンドウ高で scope-list の長さを増やしながら同意ページをレンダリングし、ボタンがビューポートから外れるタイミングを特定します。完了の条件は、交渉されたすべてのスコープが表示されたままで、リストを切り詰めることなくスクロールによって Allow ボタンと Cancel ボタンに到達できることです。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Go
- スター
- 16.6k
- フォーク
- 1.6k
- 平均マージ
- 1日 23時間
- マージ済み PR（30日）
- 543
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coder/coder のほかの issue
-
bug: AI Gateway client filter lists "Unknown" twice when NULL and literal Unknown clients coexist オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
-
feat(site): suppress the web terminal context menu when the application has enabled mouse tracking オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
bug frontend
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
bug site
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
似ている issue
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
microsoft/agent-framework-go#1179 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
[Bug]: OLLAMA_KEEP_ALIVE="5m" / "24h" crashes Ollama embedding and vision models with ValueError オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
infiniflow/ragflow#20223 · リアクション 1 件 ·
-
bug needs triage pkg/translator/faro
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
open-telemetry/opentelemetry-collector-contrib#51484 · コメント 1 件 ·