Unverified sandbox items from the Oct 11 audit: proxy checks only the TLS name, and stacked drive-letter definitions
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 38/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- typescript
調査の方向性
Read src/main/tools/net_proxy.ts:222-230 and native/sandbox-helper/src/win.rs, then verify both reported behaviors: the proxy's handling of CONNECT and TLS ClientHello extensions, and how QueryDosDeviceW definitions are checked. The issue proposes documenting the proxy limit or refusing ECH, and comparing only the first drive mapping entry; completion means both findings are verified and the chosen fixes prevent the described behavior.
索引モデルが issue の本文から書いたものです。
説明
Found in the Oct 11 audit (at c5f5641). Both need verification.
- The allow-list proxy checks only the TLS server name (
src/main/tools/net_proxy.ts:222-230). A CONNECT to an allowed host on a shared CDN can sendHost: other-siteinside TLS (domain fronting), and Encrypted Client Hello hides the real name behind the CDN's public name. Fix: document the limit, or refuse a ClientHello that carries an ECH extension (0xfe0d). DriveMapping::is_liveaccepts any entry of a drive letter's definition stack (native/sandbox-helper/src/win.rs, added in 68dd655).QueryDosDeviceWreturns every stacked definition, and.any(...)accepts the expected target even when another definition sits on top, soProjectDrive::defineskipsDefineDosDeviceWand the run starts in the wrong folder. Correctness only: access is still governed by ACLs. Fix: compare only the first entry.
- 主要言語
- TypeScript
- スター
- 2
- フォーク
- 2
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
PierrunoYT/patch のほかの issue
-
tests
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
PierrunoYT/patch#393 · コメント 1 件 ·
-
ci: macOS hardened-runtime smoke step fails because the re-signed app's helpers cannot connect (#28)オープンbug platform: macos tests
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
PierrunoYT/patch#392 · コメント 1 件 ·
-
blocked platform: windows
難易度 4/5 1〜2日 初心者へのやさしさ 52/100
PierrunoYT/patch#329 · コメント 3 件 ·
-
performance priority: low severity: low
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
PierrunoYT/patch#324 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 36/100
PierrunoYT/patch#272 · コメント 1 件 ·
PierrunoYT/patch の issue をすべて見る
似ている issue
-
Shell: serve the home app's Cookie Policy on a hard reload対応中かも @2001anshkaushik が今日担当しました。 オープンfeature module:ai
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
rocketride-org/rocketride-server#2899 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
wazuh/wazuh-dashboard-plugins#9551 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
gofish-graphics/gofish-graphics#1252 ·
メンテナーはふだん 1 日以内に返信
-
optimization P3 size:XS
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
josephfung/curia#2366 ·
メンテナーはふだん 1 日以内に返信
-
type:bug
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
nightgauge/nightgauge#2688 ·
メンテナーはふだん 1 日以内に返信