native2: client doesn't verify the host, but docs/embed.md says it does
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- swift
- Bereich
- authentication, security
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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).
- Vorherrschende Sprache
- Swift
- Sterne
- 0
- Forks
- 0
- Ø Merge
- 39 Min.
- Gemergte PRs (30 T.)
- 144
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus githubnext/desktop-tools
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
githubnext/desktop-tools#192 ·
Maintainer antworten meist innerhalb von 1 Tag
-
ace documentation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 79/100
githubnext/desktop-tools#104 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
githubnext/desktop-tools#179 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
githubnext/desktop-tools#178 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
githubnext/desktop-tools#177 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in githubnext/desktop-tools
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
swiftlang/indexstore-db#306 ·
-
host-ios R2 blocker Stability
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 70/100
paritytech/trinity-user-agents#1661 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 76/100
logto-io/logto#9837 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
area: ios S3: minor
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
manaflow-ai/cmux#24404 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag