chinux/theProx
0
0
Fork 0
Code Issues 40 Pull requests Projects Releases Packages Wiki Activity Actions

[MEDIUM] command_result/terminal-Routing ohne Owner-Bindung (Backend agent_ws + Agent vm_listener) #115

New issue
Closed
opened 2026-06-20 14:41:08 +00:00 by chinux · 3 comments
chinux commented 2026-06-20 14:41:08 +00:00
Owner
Copy link

Schweregrad: MEDIUM (durch zufaellige UUIDs gemildert, aber Owner-Bindung fehlt)

Backend (agent_ws.py:488-661): manager.resolve / terminal.data-Routing prueft nicht, dass der antwortende Agent Owner des cmd_id/session_id ist.
Agent (vm_listener.rs:181-187): command.output/result mit ungebundener cmd_id aus VM-Frame weitergereicht → koennte fremden Output verfaelschen.

Fix

  • cmd_id/session_id an node_id (bzw. owning VM) binden; eingehende Resultate nur akzeptieren, wenn der Absender der zugewiesene Owner ist.
  • Agent: nur zugewiesene cmd_ids aus dem VM-Listener akzeptieren.

Akzeptanz

  • Antwort mit fremder/unbekannter cmd_id/session_id wird verworfen.

Quelle

Code-Review 20.06.2026.

## Schweregrad: MEDIUM (durch zufaellige UUIDs gemildert, aber Owner-Bindung fehlt) **Backend** (`agent_ws.py:488-661`): `manager.resolve` / `terminal.data`-Routing prueft nicht, dass der antwortende Agent **Owner** des `cmd_id`/`session_id` ist. **Agent** (`vm_listener.rs:181-187`): `command.output/result` mit ungebundener `cmd_id` aus VM-Frame weitergereicht → koennte fremden Output verfaelschen. ## Fix - `cmd_id`/`session_id` an `node_id` (bzw. owning VM) **binden**; eingehende Resultate nur akzeptieren, wenn der Absender der zugewiesene Owner ist. - Agent: nur zugewiesene `cmd_id`s aus dem VM-Listener akzeptieren. ## Akzeptanz - Antwort mit fremder/unbekannter cmd_id/session_id wird verworfen. ## Quelle Code-Review 20.06.2026.
chinux commented 2026-06-21 07:55:50 +00:00
Author
Owner
Copy link

Backend-Owner-Bindung machbar (cmd_id→node_id in start_command tracken, resolve prüft Absender), aber der Fix umfasst laut Issue auch die Agent-Seite (vm_listener: nur zugewiesene cmd_ids). Cross-Cutting (manager + agent_ws + Node-Agent + VM-Agent). Durch zufällige UUIDs aktuell gemildert. Defer auf fokussierte Session mit Agent-Rebuild.

Backend-Owner-Bindung machbar (cmd_id→node_id in start_command tracken, resolve prüft Absender), aber der Fix umfasst laut Issue auch die Agent-Seite (vm_listener: nur zugewiesene cmd_ids). Cross-Cutting (manager + agent_ws + Node-Agent + VM-Agent). Durch zufällige UUIDs aktuell gemildert. Defer auf fokussierte Session mit Agent-Rebuild.
chinux commented 2026-06-21 08:32:18 +00:00
Author
Owner
Copy link

Umgesetzt (deployed). Owner-Bindung beidseitig:

  • Backend: cmd_owner (cmd_id→node_id) — resolve()/on_partial_output() verwerfen Antworten/Output von Nicht-Owner-Nodes. Terminal/VNC-Frames nur vom Owner-Node (node_name-Match in terminal_sessions).
  • Node-Agent 2.10.40: cmd_vmid (cmd_id→vmid) — ein VM-Agent beantwortet command_result/output nur für seine eigene vmid, kann fremde Pending-Calls nicht evicten.
    Backend gesund, Agents self-updaten. Schließe.
Umgesetzt (deployed). Owner-Bindung beidseitig: - Backend: cmd_owner (cmd_id→node_id) — resolve()/on_partial_output() verwerfen Antworten/Output von Nicht-Owner-Nodes. Terminal/VNC-Frames nur vom Owner-Node (node_name-Match in terminal_sessions). - Node-Agent 2.10.40: cmd_vmid (cmd_id→vmid) — ein VM-Agent beantwortet command_result/output nur für seine eigene vmid, kann fremde Pending-Calls nicht evicten. Backend gesund, Agents self-updaten. Schließe.
chinux commented 2026-06-26 12:20:31 +00:00
Author
Owner
Copy link

Pflicht-Begleiter fuer Public (Epic #184). Ohne cmd_id/session-Owner-Binding kann ein kompromittierter Node Ergebnisse fuer Befehle an ANDERE Nodes faelschen / fremde Sessions beanspruchen (Cross-Node-Interferenz). Die neue Challenge-Response-Auth verhindert Spoofing, aber NICHT diese Autorisierungsluecke. Muss vor Public-Betrieb erledigt sein.

**Pflicht-Begleiter fuer Public (Epic #184).** Ohne cmd_id/session-Owner-Binding kann ein kompromittierter Node Ergebnisse fuer Befehle an ANDERE Nodes faelschen / fremde Sessions beanspruchen (Cross-Node-Interferenz). Die neue Challenge-Response-Auth verhindert Spoofing, aber NICHT diese Autorisierungsluecke. Muss vor Public-Betrieb erledigt sein.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/vm-agent-update-error
fix/bulk-update-autoclose
security/tenant-scope-108
refactor/dashboard-page-split
fix/rust-agent-parity-features
feat/rust-agent-parity
archive/python-node-agent
Labels
Clear labels   
agent
Node-/VM-Agent

  
bug
Funktionaler Defekt

  
frontend
Svelte/TS Frontend

  
refactor
Umbau ohne Verhaltensaenderung

  
tech-debt
Altlast / Aufraeumen

No labels
agent
bug
frontend
refactor
tech-debt
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
chinux/theProx#115
No description provided.