native2: client doesn't verify the host, but docs/embed.md says it does
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #205 di @iamnbutler — integrata
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- swift
- Ambito
- authentication, security
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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).
- Lingua principale
- Swift
- Stelle
- 0
- Fork
- 0
- Merge medio
- 39m
- PR unite (30g)
- 162
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di githubnext/desktop-tools
-
Accessibility inspect returns far-offscreen elements, blowing the budget on long pagesForse già presa @iamnbutler l’ha presa 1 giorno fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
githubnext/desktop-tools#178 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
githubnext/desktop-tools#143 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
githubnext/desktop-tools#129 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 8/100
githubnext/desktop-tools#5 · 11 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di githubnext/desktop-tools
Issue simili
-
a:bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
vorssaint/vorssaint-utils#3527 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
rorkai/App-Store-Connect-CLI#3192 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
jellyfin/Swiftfin#2388 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P2
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
openclaw/openclaw#183929 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno