Release / verify (push) Canceled after 0s
CI / checks (push) Canceled after 0s
CI / frontend (push) Canceled after 0s
CI / backend (push) Canceled after 0s
Release / test (push) Canceled after 0s
Release / publish (backend) (push) Canceled after 0s
Release / publish (frontend) (push) Canceled after 0s
Release / publish (proxy) (push) Canceled after 0s
Release / publish (updater) (push) Canceled after 0s
- **Aus dem Testlauf 1.31.0 behoben (T2 bis T6, B15):**
- **Benutzerliste:** `GET /api/users` zeigte jedem angemeldeten Konto (auch Nur-Lese, auch per Token) alle Benutzernamen, Rollen, letzte Anmeldung und 2FA-Status.
Jetzt sehen **nur Admins** die ganze Liste, alle anderen nur das eigene Konto.
- **Hosts:** Dieselbe Adresse zweimal einzutragen wird abgelehnt (`HOST_URL_EXISTS`, auch beim Ändern der Adresse). Die Hosts-Seite und der Einrichtungsassistent
prüfen die Verbindung beim Anlegen (Schlüssel abgelehnt / zu wenig Rechte / Zertifikat / keine Verbindung), speichern bei einem Fehler nichts und bieten
„Trotzdem anlegen“. Über die API ist das `verify_connection: true` (Standard aus). **Löschen verlangt jetzt auch am Server den Hostnamen:**
`DELETE /api/hosts/{id}?confirm_name=<Name>` (sonst 409); Skripte, die Hosts löschen, müssen das anpassen.
- **Zertifikate:** `cert_type: v3_ca` wird nicht mehr angenommen (es entstand ein Zertifikat mit CA-Erweiterung, keine nutzbare CA); die Oberfläche bot es nie an.
- **Traceroute:** Zwischenstationen ohne Antwort stehen als Zeile „keine Antwort“ in der Liste (vorher sprang die Zählung 1 → 3); ein Weg ganz ohne Antwort wird erklärt.
- **Netzwerk:** Die Vorschau prüft Zahlenbereiche vorab (VLAN-Tag 1–4094, PCP 0–7, VHID 1–255, Basis 1–254, Skew 0–254, gegen das Modell der OPNsense gelesen): `422` mit dem
Feldnamen statt `502` von der Firewall.
- **Gruppen:** Eine Gruppe, die ein VPN-Objekt als Ziel hat, lässt sich nicht mehr löschen (die Verknüpfung fiel still weg).
- Das Skript `scripts/verify_i18n.mjs` läuft jetzt auch gegen die HTTPS-Test-Instanz.
- **Regel-Bereinigung: OPNexus selbst ist automatisch geschützt (B10).** Beim Planen fragt die Bereinigung jede Firewall, von welcher Adresse sie OPNexus kommen sieht
(die eigene API-Verbindung in der Zustandstabelle), und schützt diese Adresse zusätzlich zu den Verwaltungsquellen. Vorher konnte eine Regel abgeschaltet werden, über
die nur OPNexus die Firewall erreicht, wenn in den Quellen nur Arbeitsplätze standen; dann scheiterte auch das automatische Zurücksetzen. Die Vorschau zeigt die Adresse
je Firewall (nur Admins) oder sagt, dass sie sich nicht ermitteln ließ (API-Recht „Diagnose: Zustände“, Management-Tunnel).
- **Keine verlorenen Regeltreffer mehr (B11):** Jedes Anwenden setzt die Regelzähler der Firewall zurück. OPNexus liest sie jetzt unmittelbar vor jedem Anwenden, das es
selbst auslöst (Regeln, NAT, Aliase, IPsec/OpenVPN/WireGuard, Netzwerk); vorher konnte eine Regel deshalb fälschlich als „ungenutzt“ gelten.
- **Einstellungen › Regel-Bereinigung:** schlägt die eigene Browser-Adresse als Verwaltungsquelle vor (B12). Geprüft: OPNsense 26.7 hat kein „savepoint“ zum Zurücknehmen
durch die Firewall selbst (Quellcode und live 404), das automatische Zurücksetzen bleibt bei OPNexus.
- **Installation unter Windows:** Neue Anleitung auf der Website (`/docs/windows/`, auch auf Englisch) und im Handbuch: Docker Desktop mit WSL 2, Zertifizierungsstelle importieren, Zugriff aus dem Netz (Windows-Firewall), Updates mit `opnexus` in WSL und Fehlerbehebung. Entwickelt und getestet wird weiterhin unter Linux; die Anleitung sagt das offen.
- **Sicherheit und Stabilität (aus einer externen Prüfung):**
- **Rückweg bleibt erhalten:** Wer nach „Konfiguration geschrieben, Anwenden fehlgeschlagen“ erneut ausrollte, überschrieb den ursprünglichen Rückweg; die automatische
Rücknahme stellte dann den unbestätigten Stand wieder her. Jetzt bleibt der Rückweg der Stand vor dem ersten Versuch. Wartung, Regel-Bereinigung und
„Konfiguration als Code“ behandeln eine solche Zeile wie eine offene Bestätigung. Gilt für Regeln, Aliase, Port-Forwards, 1:1-NAT, Outbound-NAT und Hierarchie-Regeln.
- **Anmeldung und 2FA:** Eine Passwort-Anmeldung, die genau während der Aktivierung von 2FA lief, bekam eine Sitzung ohne zweiten Faktor. Jetzt wird sie mit `TOTP_REQUIRED` abgewiesen.
- **Letzter Administrator:** Zwei Administratoren, die sich gleichzeitig herabstuften oder gegenseitig löschten, konnten alle Administratoren entfernen. Beides ist jetzt serialisiert.
- **Operatoren:** Mit ausgeschaltetem „Operatoren dürfen ausrollen“ konnten sie noch Snapshots (Boot-Umgebungen) anlegen. Ein Test hält künftig jede schreibende Route
fest, die keiner Rechte-Regel zugeordnet ist.
- **Zertifikate und ACME:** Der Ziel-Host ist nach dem ersten Push fest (`409 ZIEL_HOST_NACH_AUSROLLEN_FEST`); vorher meldete ein Host-Wechsel Erfolg für einen Host, der nie
etwas erhielt. Ein ACME-Konto mit Anfragen lässt sich nicht auf einen anderen Host verschieben (`ACME_KONTO_WIRD_VON_ANFRAGEN_VERWENDET`).
- **VPN-Ausrollen:** IPsec, OpenVPN und WireGuard sichern jede von der Firewall vergebene Kennung sofort; bei einem Abbruch entstehen beim nächsten Versuch keine Duplikate.
Pro VPN läuft nur ein Ausrollen oder Entfernen zugleich (`409 VPN_DEPLOY_LAEUFT_BEREITS`).
- **Konfiguration als Code:** Gültiges YAML mit falsch geformtem Abschnitt (z. B. `aliases: 42`) ergibt `422 CONFIG_INVALID` statt eines Serverfehlers.
- **`.env`:** Die Anleitungen legen sie mit `install -m 600` an; `opnexus update` und `opnexus status` setzen eine zu offene Datei auf 0600. `opnexus rollback` liest seinen
Update-Zustand nur noch als Text und führt ihn nie aus.
- **Abhängigkeiten:** Next.js 15.5.28, PyJWT 2.15.1.
153 lines
7.3 KiB
Python
153 lines
7.3 KiB
Python
"""Separation of duties: with ``operators_may_deploy`` off, only admins roll changes out (AuthMiddleware enforces it).
|
|
|
|
The rule-cleanup settings (1.31) live here too: how many rules one cleanup may touch, how long a rule must stay
|
|
switched off before it may be deleted, the management sources (addresses the firewalls are managed from; no rule
|
|
that could match them is ever cleaned up) and extra management ports (besides the API port, 443, 80 and 22). Only an admin writes any of it (middleware: ``/api/settings/...`` is
|
|
admin-only for operators and tokens below scope admin; checked again here) and every change is audited with the
|
|
old and the new values."""
|
|
import ipaddress
|
|
from typing import Any
|
|
|
|
from fastapi import APIRouter, Depends, Request
|
|
from pydantic import BaseModel, Field
|
|
from sqlalchemy.ext.asyncio import AsyncSession
|
|
|
|
from app.database import get_db
|
|
from app.errors import ErrorCode, api_error
|
|
from app.models.governance import GovernanceSettings
|
|
from app.models.user import AdminUser
|
|
from app.routers.auth import client_ip, current_user
|
|
from app.services import audit_service
|
|
|
|
router = APIRouter(prefix="/settings/governance", tags=["governance"])
|
|
|
|
MAX_CLEANUP_RULES = 25 # same ceiling as services/rule_cleanup_service.MAX_RULES_CEILING
|
|
MAX_DELETE_MIN_DAYS = 365
|
|
MAX_MANAGEMENT_SOURCES = 20
|
|
MAX_MANAGEMENT_PORTS = 20
|
|
|
|
|
|
class GovernanceRead(BaseModel):
|
|
operators_may_deploy: bool
|
|
cleanup_max_rules: int = 10
|
|
cleanup_delete_min_days: int = 7
|
|
# The list itself is shown to admins only; everybody else sees just whether it is set.
|
|
cleanup_management_sources: list[str] = Field(default_factory=list)
|
|
cleanup_management_sources_configured: bool = False
|
|
# Extra management ports (besides API port, 443, 80, 22): also admins only, others see the flag.
|
|
cleanup_management_ports: list[int] = Field(default_factory=list)
|
|
cleanup_management_ports_configured: bool = False
|
|
# The address this request came from, as a suggestion for the management sources (B12; admins only). Only a
|
|
# suggestion: the firewalls may see the workstation under another address (NAT, VPN).
|
|
your_address: str | None = None
|
|
|
|
|
|
class GovernanceUpdate(BaseModel):
|
|
operators_may_deploy: bool | None = None
|
|
cleanup_max_rules: int | None = None
|
|
cleanup_delete_min_days: int | None = None
|
|
cleanup_management_sources: list[str] | None = None
|
|
cleanup_management_ports: list[Any] | None = None # validated in clean_management_ports (no bool coercion)
|
|
|
|
|
|
def _bad(field: str, reason: str):
|
|
return api_error(422, ErrorCode.BEREINIGUNG_EINSTELLUNG_UNGUELTIG, field=field, reason=reason)
|
|
|
|
|
|
def clean_management_sources(values: list[str]) -> list[str]:
|
|
"""IP addresses and CIDR networks only (no host names: they could resolve differently later), canonical form,
|
|
without duplicates; not the whole address space, which would protect every rule."""
|
|
if len(values) > MAX_MANAGEMENT_SOURCES:
|
|
raise _bad("cleanup_management_sources", "too_many")
|
|
cleaned: list[str] = []
|
|
for raw in values:
|
|
text = str(raw).strip()
|
|
try:
|
|
net = ipaddress.ip_network(text, strict=False)
|
|
except ValueError:
|
|
raise _bad("cleanup_management_sources", "not_an_address") from None
|
|
if net.prefixlen == 0:
|
|
raise _bad("cleanup_management_sources", "everything")
|
|
entry = str(net.network_address) if net.prefixlen == net.max_prefixlen else str(net)
|
|
if entry not in cleaned:
|
|
cleaned.append(entry)
|
|
return cleaned
|
|
|
|
|
|
def clean_management_ports(values) -> list[int]:
|
|
"""TCP ports 1..65535, at most MAX_MANAGEMENT_PORTS, without duplicates, sorted."""
|
|
if len(values) > MAX_MANAGEMENT_PORTS:
|
|
raise _bad("cleanup_management_ports", "too_many")
|
|
ports: set[int] = set()
|
|
for raw in values:
|
|
text = str(raw).strip()
|
|
if isinstance(raw, bool) or not text.isdigit() or not 1 <= int(text) <= 65535:
|
|
raise _bad("cleanup_management_ports", "not_a_port")
|
|
ports.add(int(text))
|
|
return sorted(ports)
|
|
|
|
|
|
def _read(row: GovernanceSettings, user: AdminUser) -> GovernanceRead:
|
|
sources = list(row.cleanup_management_sources or [])
|
|
ports = list(row.cleanup_management_ports or [])
|
|
return GovernanceRead(
|
|
operators_may_deploy=row.operators_may_deploy, cleanup_max_rules=row.cleanup_max_rules or 10,
|
|
cleanup_delete_min_days=row.cleanup_delete_min_days or 7,
|
|
cleanup_management_sources=sources if user.role == "admin" else [],
|
|
cleanup_management_sources_configured=bool(sources),
|
|
cleanup_management_ports=ports if user.role == "admin" else [],
|
|
cleanup_management_ports_configured=bool(ports))
|
|
|
|
|
|
async def get_settings(db: AsyncSession) -> GovernanceSettings:
|
|
row = await db.get(GovernanceSettings, 1)
|
|
if row is None:
|
|
row = GovernanceSettings(id=1, operators_may_deploy=True)
|
|
db.add(row)
|
|
await db.flush()
|
|
return row
|
|
|
|
|
|
def _suggestion(request: Request, user: AdminUser) -> str | None:
|
|
if user.role != "admin":
|
|
return None
|
|
try:
|
|
return str(ipaddress.ip_address(client_ip(request)))
|
|
except ValueError:
|
|
return None
|
|
|
|
|
|
@router.get("", response_model=GovernanceRead)
|
|
async def read(request: Request, db: AsyncSession = Depends(get_db), user: AdminUser = Depends(current_user)):
|
|
"""Readable by everybody signed in (the UI explains why a deploy was refused)."""
|
|
return _read(await get_settings(db), user).model_copy(update={"your_address": _suggestion(request, user)})
|
|
|
|
|
|
@router.put("", response_model=GovernanceRead)
|
|
async def update(payload: GovernanceUpdate, request: Request, db: AsyncSession = Depends(get_db), user: AdminUser = Depends(current_user)):
|
|
if user.role != "admin":
|
|
raise api_error(403, ErrorCode.ADMIN_REQUIRED)
|
|
row = await get_settings(db)
|
|
changes: dict[str, dict] = {}
|
|
if payload.cleanup_max_rules is not None:
|
|
if not 1 <= payload.cleanup_max_rules <= MAX_CLEANUP_RULES:
|
|
raise _bad("cleanup_max_rules", f"1..{MAX_CLEANUP_RULES}")
|
|
if payload.cleanup_delete_min_days is not None:
|
|
if not 1 <= payload.cleanup_delete_min_days <= MAX_DELETE_MIN_DAYS:
|
|
raise _bad("cleanup_delete_min_days", f"1..{MAX_DELETE_MIN_DAYS}")
|
|
sources = clean_management_sources(payload.cleanup_management_sources) if payload.cleanup_management_sources is not None else None
|
|
ports = clean_management_ports(payload.cleanup_management_ports) if payload.cleanup_management_ports is not None else None
|
|
for field, value in (("cleanup_max_rules", payload.cleanup_max_rules), ("cleanup_delete_min_days", payload.cleanup_delete_min_days),
|
|
("cleanup_management_sources", sources), ("cleanup_management_ports", ports)):
|
|
if value is not None and value != getattr(row, field):
|
|
changes[field] = {"old": getattr(row, field), "new": value}
|
|
setattr(row, field, value)
|
|
if payload.operators_may_deploy is not None:
|
|
row.operators_may_deploy = payload.operators_may_deploy
|
|
if changes:
|
|
db.add(audit_service.make_event(
|
|
kind="settings", action="cleanup_settings", status="accepted", entity_id="governance",
|
|
message="Einstellungen der Regel-Bereinigung geändert", details={"changes": changes}))
|
|
await db.commit()
|
|
return _read(row, user).model_copy(update={"your_address": _suggestion(request, user)})
|