Unverified sandbox items from the Oct 11 audit: proxy checks only the TLS name, and stacked drive-letter definitions
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2
- Forks
- 2
- Merge medio
- 8 h 28 min
- PR fusionados (30 d)
- 26
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de PierrunoYT/patch
-
enhancement priority: low severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
PierrunoYT/patch#305 ·
Los mantenedores suelen responder en 1 día
-
security: isAppPageUrl checks devRendererUrl with startsWith instead of comparing parsed URL originsAbiertopriority: low security severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
PierrunoYT/patch#300 ·
Los mantenedores suelen responder en 1 día
-
Claude Code chats: tool calls made inside Claude Code subagents leave cards that end as "(stopped)"Abiertobug priority: low severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
PierrunoYT/patch#289 ·
Los mantenedores suelen responder en 1 día
-
bug priority: medium severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
PierrunoYT/patch#285 ·
Los mantenedores suelen responder en 1 día
-
priority: medium security severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
PierrunoYT/patch#283 ·
Los mantenedores suelen responder en 1 día
Todos los issues de PierrunoYT/patch
Issues similares
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
lingdojo/kana-dojo#32663 · 1 comentario · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
Admin: Title Separator field can't be cleared (input falls back to "|")Posiblemente ocupada @emdashbot la tomó hoy. Abiertobot:bug bot:in-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
emdash-cms/emdash#4236 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
PHP workflow streaming requests buffer the full SSE responsePosiblemente ocupada @Winoren la tomó hoy. Abiertoreview: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
langgenius/dify#44270 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
area:holodeck review-finding
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
hyperdrive-technology/holodeck#142 ·
Los mantenedores suelen responder en 1 día
-
session-worker-compile.yml uses a floating oven-sh/setup-bun@v2 tag instead of the pinned SHAAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
code-yeongyu/senpi#3573 ·
Los mantenedores suelen responder en 1 día