[HIGH] /api/audit/commands ohne Tenant-Scope — globale CommandLog-Historie (params/output) #110

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/audit/commands (audit_router.py:76-99) liefert CommandLog.to_dict() (action+params+output) aller Mandanten; params kann IPs/Tokens/Pfade enthalten. Kein Scope, nur get_current_user.

Fix: Scope-Muster aus #108; require_read(). Akzeptanz: nur eigene Nodes; Cross-Tenant-Test gruen.
Verifiziert @ 282b76e.

Teil von #108. **HIGH.** `GET /api/audit/commands` (`audit_router.py:76-99`) liefert `CommandLog.to_dict()` (action+params+output) **aller Mandanten**; `params` kann IPs/Tokens/Pfade enthalten. Kein Scope, nur `get_current_user`. **Fix:** Scope-Muster aus #108; `require_read()`. **Akzeptanz:** nur eigene Nodes; Cross-Tenant-Test gruen. 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/audit/commands (#110)

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 8d1a5cb (Branch security/tenant-scope-108). Test: server/tests/test_tenant_scope.py::test_audit_commands_scope. Nicht geschlossen — Review durch Sebastian.

Umgesetzt in `8d1a5cb` (Branch `security/tenant-scope-108`). Test: `server/tests/test_tenant_scope.py::test_audit_commands_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/audit/commands (#110)

chinux commented

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

Author

Owner

Tenant-Scope umgesetzt in 49d29a4 (in main, deployed). Test test_audit_commands_scope. Schließe als erledigt.

Tenant-Scope umgesetzt in `49d29a4` (in main, deployed). Test `test_audit_commands_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:13 +00:00

Author

Owner

Verifiziert @ 6fabc1f → gefixt, wird geschlossen.

audit_router.py /commands: require_read() + Scope-Filter (eigene Nodes bei leerem node_name, 403 bei fremdem).

Muster aus #108 korrekt umgesetzt.

**Verifiziert @ 6fabc1f → gefixt, wird geschlossen.** audit_router.py `/commands`: `require_read()` + Scope-Filter (eigene Nodes bei leerem node_name, 403 bei fremdem). Muster aus #108 korrekt umgesetzt.