[HIGH] /api/command/logs ohne Tenant-Scope — Befehls-Output aller Nodes lesbar #109

New issue

Closed

opened 2026-06-20 14:40:13 +00:00 by chinux · 3 comments

chinux commented

2026-06-20 14:40:13 +00:00

Owner

Teil von #108. HIGH. GET /api/command/logs (command_router.py:244-264) gibt CommandExecutionLog.output (bis 50k Zeichen STDOUT) + vollen command aller Nodes zurueck — nur get_current_user, kein check_node_scope, kein Scope-Filter. Befehls-Output kann Secrets enthalten.

Fix: Scope-Muster aus #108; require_read(). Akzeptanz: User sieht nur Logs eigener Nodes; leerer node_name ⇒ eigene Nodes.
Verifiziert @ 282b76e.

Teil von #108. **HIGH.** `GET /api/command/logs` (`command_router.py:244-264`) gibt `CommandExecutionLog.output` (bis 50k Zeichen STDOUT) + vollen `command` **aller Nodes** zurueck — nur `get_current_user`, kein `check_node_scope`, kein Scope-Filter. Befehls-Output kann Secrets enthalten. **Fix:** Scope-Muster aus #108; `require_read()`. **Akzeptanz:** User sieht nur Logs eigener Nodes; leerer `node_name` ⇒ eigene Nodes. Verifiziert @ 282b76e.

chinux added the

bug

label

2026-06-20 14:40:13 +00:00

chinux referenced this issue from a commit

2026-06-20 15:09:57 +00:00

fix(security): tenant-scope /api/command/logs (#109)

chinux referenced this issue

2026-06-20 15:13:27 +00:00

[CRITICAL] Mandantentrennung: 5 History-/Log-Endpoints ohne Tenant-Scope (Cross-Tenant-Leak) #108

chinux commented

2026-06-20 15:13:43 +00:00

Author

Owner

Umgesetzt in fb12f7d (Branch security/tenant-scope-108). Test: server/tests/test_tenant_scope.py::test_command_logs_scope. Nicht geschlossen — Review durch Sebastian.

Umgesetzt in `fb12f7d` (Branch `security/tenant-scope-108`). Test: `server/tests/test_tenant_scope.py::test_command_logs_scope`. Nicht geschlossen — Review durch Sebastian.

chinux referenced this issue from a commit

2026-06-20 20:58:40 +00:00

fix(security): tenant-scope /api/command/logs (#109)

chinux commented

2026-06-20 20:59:36 +00:00

Author

Owner

Tenant-Scope umgesetzt in 2608722 (in main, deployed). Test test_command_logs_scope. Schließe als erledigt.

Tenant-Scope umgesetzt in `2608722` (in main, deployed). Test `test_command_logs_scope`. Schließe als erledigt.

chinux closed this issue

2026-06-20 20:59:36 +00:00

chinux referenced this issue

2026-06-20 20:59:36 +00:00

[CRITICAL] Mandantentrennung: 5 History-/Log-Endpoints ohne Tenant-Scope (Cross-Tenant-Leak) #108

chinux commented

2026-06-23 22:01:11 +00:00

Author

Owner

Verifiziert @ 6fabc1f → gefixt, wird geschlossen.

command_router.py /logs: require_read(), list_user_accessible_nodes, Scope-WHERE vor LIMIT, leerer node_name ⇒ nur eigene Nodes.

Muster aus #108 korrekt umgesetzt.

**Verifiziert @ 6fabc1f → gefixt, wird geschlossen.** command_router.py `/logs`: `require_read()`, `list_user_accessible_nodes`, Scope-WHERE vor LIMIT, leerer node_name ⇒ nur eigene Nodes. Muster aus #108 korrekt umgesetzt.