native2: client doesn't verify the host, but docs/embed.md says it does
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- swift
- Área
- authentication, security
Línea de trabajo
Read DesktopHost/Server.swift:148 and compare its peer check with the former client check in native/sources/client/client.swift; review the current client entry points named in the issue: the client CLI, desktop-tools-mcp, and DesktopAppLink. Check docs/embed.md:24 against the intended behavior. Done means either host signature verification is restored on the client side or the documentation explains why it was deliberately dropped.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In native2 only the host checks its peer: DesktopHost/Server.swift:148 admits a client when Peer.satisfies matches the team and exact identifier. Neither the client CLI, desktop-tools-mcp nor DesktopAppLink checks the host's signature before sending requests; they exchange only a version hello.
The Peekaboo-based client did check (native/sources/client/client.swift, trustedHostTeamIDs), and docs/embed.md:24 still says "the client verifies the host's team".
The risk is that a process at the socket path that isn't our host gets the agent's requests, and can lie about results.
Either:
- add the reverse check:
Peer(socket:)on the client side, requiring the host to be signed by the same team (asownSigning()already does for the host), or - if dropping it was deliberate, update
docs/embed.mdto say so and why.
Found while updating the Desktop Tools write-up for native2 (checked at 0d89ab5).
- Lenguaje dominante
- Swift
- Estrellas
- 0
- Forks
- 0
- Merge medio
- 39 min
- PR fusionados (30 d)
- 144
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 githubnext/desktop-tools
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
githubnext/desktop-tools#192 ·
Los mantenedores suelen responder en 1 día
-
Ace: adopt desktopToolRules()Abiertoace documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
githubnext/desktop-tools#104 ·
Los mantenedores suelen responder en 1 día
-
Status-item NSMenu (MenuBarExtra .menu): opens but can't be inspected via Accessibility or dismissedAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
githubnext/desktop-tools#193 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
githubnext/desktop-tools#179 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
githubnext/desktop-tools#178 ·
Los mantenedores suelen responder en 1 día
Todos los issues de githubnext/desktop-tools
Issues similares
-
product / auth product / databases product / functions product / messaging product / storage product / vcs
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
storage: eliminate force-unwraps and fatalError in continueStreamingUploadPosiblemente ocupada @chingor13 la tomó hoy. Abiertoapi: storage type: cleanup
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
googleapis/google-cloud-swift#1527 ·
Los mantenedores suelen responder en 1 día
-
ci-failure ci:assertion test
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
openwatersio/slackwater-ios#811 ·
Los mantenedores suelen responder en 1 día
-
ai-assisted
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
swiftlang/swift-testing#1979 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
Los mantenedores suelen responder en 1 día