ChatGPT sign-in: the redirect URI says localhost but the callback server binds only 127.0.0.1
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 55/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- node.js, typescript
調査の方向性
Start with src/main/llm/codex_auth.ts, especially the redirect URI at line 23 and server binding at line 244. Read the OAuth callback setup and check how the server reports bind failures; the issue proposes supporting both 127.0.0.1 and ::1 and refusing to start if either is occupied. Done means sign-in callbacks work with either localhost address and a local port conflict is reported.
索引モデルが issue の本文から書いたものです。
説明
From Finding 5 of the earlier SECURITY-REVIEW.md and the Oct 7 audit, checked against the current code.
Problem
The OAuth redirect is http://localhost:1455/auth/callback (src/main/llm/codex_auth.ts:23), but the server listens only on 127.0.0.1 (codex_auth.ts:244). Browsers may resolve localhost to ::1 first. Another local process listening on [::1]:1455 would then get the callback instead of Patch, or sign-in would fail. PKCE (S256) and the random state mean an intercepted code can't be exchanged, so the realistic impact is a failed or blocked sign-in, not token theft. The port is fixed by the provider's client registration.
Proposal
Listen on both 127.0.0.1 and ::1 (two servers, or one with ipv6Only: false on ::), and refuse to start if either is taken. Document that a local process can block sign-in.
- 主要言語
- TypeScript
- スター
- 2
- フォーク
- 2
- 平均マージ
- 5時間 28分
- マージ済み PR（30日）
- 24
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
PierrunoYT/patch のほかの issue
-
enhancement priority: low security severity: low
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
PierrunoYT/patch#260 ·
メンテナーはふだん 1 日以内に返信
-
bug priority: low severity: low
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
PierrunoYT/patch#259 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
platform: macos priority: medium security severity: low tests
難易度 4/5 3〜5日 初心者へのやさしさ 15/100
PierrunoYT/patch#257 ·
メンテナーはふだん 1 日以内に返信
-
bug platform: windows priority: medium severity: medium
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
PierrunoYT/patch#242 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug priority: high severity: high
難易度 4/5 3〜5日 初心者へのやさしさ 42/100
PierrunoYT/patch#240 ·
メンテナーはふだん 1 日以内に返信
PierrunoYT/patch の issue をすべて見る
似ている issue
-
[Docs] README: FAQ setup command, IDA in the intro, Node badge対応中かも @akram1089 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
[Feature]: [P3] engine-rs: the package source hash should ignore line endings and untracked filesオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
maniator/verticopolis#880 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
siyuan-note/siyuan#20353 ·
メンテナーはふだん 1 日以内に返信
-
afk-ok area:data-quality importer size:S
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
enorm-labs/event-junkie#3027 ·
メンテナーはふだん 1 日以内に返信