diff --git a/apps/tesm/pyproject.toml b/apps/tesm/pyproject.toml index 82a9bfc..997ac34 100644 --- a/apps/tesm/pyproject.toml +++ b/apps/tesm/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "tesm" -version = "2.1.2" +version = "2.2.0" description = "TESM -- Verwaltung und Ueberwachung von PoE-Endgeraeten und Aruba-Switchen" requires-python = ">=3.11" dependencies = [ diff --git a/apps/tesm/src/tesm/blueprints/fileshare.py b/apps/tesm/src/tesm/blueprints/fileshare.py index 076da6e..4d86727 100644 --- a/apps/tesm/src/tesm/blueprints/fileshare.py +++ b/apps/tesm/src/tesm/blueprints/fileshare.py @@ -18,6 +18,7 @@ from flask import ( from tesm_core import audit from tesm_core.auth.models import current_user, permission_required from tesm_core.auth.service import client_ip +from tesm_core import uploads from tesm_core.extension import core, db, transaction from ..services import fileshare as service @@ -75,7 +76,10 @@ def browse(): token=token, listing=listing, error=error, - max_upload_mb=int(core().settings.get(conn, "fileshare_upload_max_mb") or 512), + # Dieselbe Zahl, die auch die Anfrage begrenzt und in nginx steht -- + # vorher versprach diese Seite 512 MB, waehrend bei 64 MB abgebrochen + # wurde. + max_upload_mb=uploads.limit_mb(conn), ) @@ -95,7 +99,12 @@ def download(token: str): path, as_attachment=True, download_name=path.name, - mimetype=service.download_mime(path.name), + # Bewusst der neutrale Typ und nicht der erkannte: iOS Safari + # entscheidet nach dem MIME-Typ und zeigt eine "application/pdf" im + # Browser an, auch wenn sie als Anhang ausgeliefert wird. Wer hier + # klickt, will die Datei aber ablegen -- angesehen wird sie in der + # Vorschau. Fuer alle anderen Browser aendert sich damit nichts. + mimetype="application/octet-stream", ) @@ -148,7 +157,7 @@ def upload(token: str): flash("Keine Datei ausgewaehlt.", "danger") return redirect(url_for("fileshare.browse", freigabe=token, pfad=relative)) - limit = int(core().settings.get(db(), "fileshare_upload_max_mb") or 512) * 1024 * 1024 + limit = uploads.limit_bytes(db()) stored = 0 try: target_dir = service.resolve_path(mount, relative) @@ -172,6 +181,41 @@ def upload(token: str): return redirect(url_for("fileshare.browse", freigabe=token, pfad=relative)) +@bp.post("//formular") +@permission_required("fileshare.create") +def save_form(token: str): + """Nimmt ein in der Vorschau ausgefuelltes PDF und legt es daneben ab. + + Als **neue** Datei: auf einer Freigabe liegt in der Regel die leere + Vorlage, und die soll leer bleiben. Wer das Original ersetzen will, kann + die alte Datei danach loeschen -- das ist eine bewusste Handlung. + """ + mount = _mount_or_404(token) + relative = request.form.get("pfad") or "" + upload_file = request.files.get("datei") + if upload_file is None or not upload_file.filename: + return {"error": "Es kam keine Datei an."}, 400 + + grenze = uploads.limit_bytes(db()) + daten = upload_file.stream.read(grenze + 1) + if len(daten) > grenze: + return {"error": f"Zu gross (Grenze {grenze // 1024 // 1024} MB)."}, 413 + if not daten.startswith(b"%PDF-"): + # Diese Route nimmt nur, was pdf.js hier ablegt. Ein anderer Inhalt + # waere ein Umweg, um beliebige Dateien in die Freigabe zu bekommen. + return {"error": "Das ist keine PDF-Datei."}, 415 + + try: + original = service.resolve_path(mount, relative) + ziel = service.free_name(original.parent, original.stem, "-ausgefuellt", ".pdf") + ziel.write_bytes(daten) + except service.FileshareError as exc: + return {"error": str(exc)}, 400 + + _audit("fileshare.form_saved", f"{mount.label}:{relative}", ziel.name, severity="notice") + return {"message": f"Als {ziel.name} gespeichert."} + + @bp.post("//ordner") @permission_required("fileshare.create") def make_directory(token: str): diff --git a/apps/tesm/src/tesm/services/fileshare.py b/apps/tesm/src/tesm/services/fileshare.py index 79b1834..ff0ca43 100644 --- a/apps/tesm/src/tesm/services/fileshare.py +++ b/apps/tesm/src/tesm/services/fileshare.py @@ -57,6 +57,10 @@ PREVIEW_MIME: dict[str, str] = { ".gif": "image/gif", ".webp": "image/webp", ".bmp": "image/bmp", + # Das Format, in dem ein iPad fotografiert. Safari zeigt es an, andere + # Browser nicht -- die Vorschau sagt es dann und bietet den Download an. + ".heic": "image/heic", + ".heif": "image/heif", ".txt": "text/plain; charset=utf-8", ".csv": "text/plain; charset=utf-8", ".log": "text/plain; charset=utf-8", @@ -68,6 +72,21 @@ PREVIEW_MIME: dict[str, str] = { #: im Ursprung der Anwendung ausgefuehrt. Download bleibt moeglich. +def preview_kind(name: str) -> str: + """Welche Anzeige passt: ``bild``, ``pdf`` oder ``text``? + + Die Unterscheidung gehoert hierher und nicht in die Vorlage: die + Zuordnung Endung -> MIME-Typ steht schon hier, und eine zweite Liste in + Jinja liefe frueher oder spaeter auseinander. + """ + mime = PREVIEW_MIME.get(Path(name).suffix.lower(), "") + if mime.startswith("image/"): + return "bild" + if mime == "application/pdf": + return "pdf" + return "text" + + class FileshareError(Exception): """Fehler mit Klartextmeldung fuer die Oberflaeche.""" @@ -528,6 +547,7 @@ def list_directory(mount: Mount, relative: str = "") -> dict[str, Any]: directories.append(record) else: record["previewable"] = entry.suffix.lower() in PREVIEW_MIME + record["preview_kind"] = preview_kind(entry.name) files.append(record) except OSError as exc: raise FileshareError(f"Verzeichnis nicht lesbar ({exc}).") from exc @@ -568,6 +588,23 @@ def safe_filename(name: str) -> str: return cleaned +def free_name(parent: Path, stem: str, zusatz: str, suffix: str) -> Path: + """Ein freier Name neben dem Original: ``vorlage-ausgefuellt.pdf``. + + Nichts ueberschreiben: auf einer Freigabe liegt meist die leere Vorlage, + und zwei Leute fuellen sie nacheinander aus. Ab dem zweiten Mal wird + durchnummeriert. + """ + kandidat = parent / f"{stem}{zusatz}{suffix}" + zaehler = 2 + while kandidat.exists(): + kandidat = parent / f"{stem}{zusatz}-{zaehler}{suffix}" + zaehler += 1 + if zaehler > 200: + raise FileshareError("Zu viele Dateien dieses Namens -- bitte aufraeumen.") + return kandidat + + def delete_entry(mount: Mount, relative: str) -> str: path = resolve_path(mount, relative) if path == mount.path.resolve(): diff --git a/apps/tesm/src/tesm/settings_spec.py b/apps/tesm/src/tesm/settings_spec.py index 28983a3..61b244f 100644 --- a/apps/tesm/src/tesm/settings_spec.py +++ b/apps/tesm/src/tesm/settings_spec.py @@ -79,15 +79,6 @@ TESM_SETTINGS: tuple[SettingSpec, ...] = ( minimum=1, maximum=168, ), - SettingSpec( - "fileshare_upload_max_mb", - 512, - "int", - "Maximale Dateigroesse beim Hochladen (MB)", - section="fileshare", - minimum=1, - maximum=8192, - ), # -- Wartung ------------------------------------------------------------ # SettingSpec( "maintenance_command_timeout", diff --git a/apps/tesm/src/tesm/templates/tesm/fileshare.html b/apps/tesm/src/tesm/templates/tesm/fileshare.html index faf9463..da851e1 100644 --- a/apps/tesm/src/tesm/templates/tesm/fileshare.html +++ b/apps/tesm/src/tesm/templates/tesm/fileshare.html @@ -103,6 +103,8 @@ @@ -201,14 +205,40 @@ Dialog für alle Dateien statt einer je Zeile. #} {% call modal('file-preview', 'Vorschau', '', 'wide') %}
-
- + {# Drei Anzeigen, eine je Art. Ein Rahmen für alles zeigte ein Foto vom + iPad in Originalgrösse (also einen Ausschnitt) und ein PDF auf iPadOS + nur als Standbild -- Formularfelder liessen sich nicht anfassen. #} +
+
+ + + + +
{{ icon('download', size=15) }} Herunterladen + {% if current_user.has('fileshare.create') %} + + {% endif %}
{% endcall %} diff --git a/deploy/install.sh b/deploy/install.sh index ea21824..e673898 100644 --- a/deploy/install.sh +++ b/deploy/install.sh @@ -190,7 +190,10 @@ fi # nicht ausdruecklich neu gesetzt wird. ENV_FILE="$ETC_TESM/$NAME.env" # Wo die Daten liegen, steht in der Umgebungsdatei; ohne sie beim Vorgabeort. -DATEN_DIR="$(sed -n "s/^${ENV_PREFIX}DATA_DIR=//p" "$ENV_FILE" 2>/dev/null | tail -1)" +# "|| true": bei einer Neuinstallation gibt es die Datei noch nicht, und ein +# scheiterndes sed in einer Zuweisung beendet unter "set -e" das ganze Skript. +DATEN_DIR="$(sed -n "s/^${ENV_PREFIX}DATA_DIR=//p" "$ENV_FILE" 2>/dev/null || true)" +DATEN_DIR="$(printf '%s' "$DATEN_DIR" | tail -1)" DATEN_DIR="${DATEN_DIR:-$ROOT/data}" if [[ -z "$PORT" && -f "$ENV_FILE" ]]; then PORT="$(sed -n "s/^${ENV_PREFIX}BIND=127\.0\.0\.1:\([0-9]\{1,5\}\)$/\1/p" "$ENV_FILE" | head -1)" diff --git a/packages/tesm-core/pyproject.toml b/packages/tesm-core/pyproject.toml index de32f0a..bbbe41e 100644 --- a/packages/tesm-core/pyproject.toml +++ b/packages/tesm-core/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "tesm-core" -version = "2.1.2" +version = "2.2.0" description = "Gemeinsames Anwendungsgeruest (Auth, RBAC, Navigation, Design-System, Migrationen) fuer TESM und TESM-Lizenzserver" requires-python = ">=3.11" dependencies = [ diff --git a/packages/tesm-core/src/tesm_core/app_factory.py b/packages/tesm-core/src/tesm_core/app_factory.py index 9283ee5..b7d3cdf 100644 --- a/packages/tesm-core/src/tesm_core/app_factory.py +++ b/packages/tesm-core/src/tesm_core/app_factory.py @@ -31,6 +31,7 @@ from .security import csrf from .security import sessions as session_store from .security.sessions import SqliteSessionInterface from .settings_store import SettingSpec, SettingsStore +from . import uploads from .sysops import SysOps from .auth import directory from .auth import routes as auth_routes @@ -134,7 +135,7 @@ def create_base_app( #: Wortmarke, ohne Zeile darunter; die Marke selbst zeigen beide #: Anwendungen. auth_tagline: str = "", - app_version: str = "2.1.2", + app_version: str = "2.2.0", built_in_trusted_keys: tuple[str, ...] = (), enable_license: bool = True, template_folder: str | None = None, @@ -163,6 +164,10 @@ def create_base_app( config.data_dir.mkdir(parents=True, exist_ok=True) config.log_dir.mkdir(parents=True, exist_ok=True) + # Die Groessengrenze einer Anfrage kommt aus der Einstellung, nicht aus + # der Konfiguration beim Start. + app.request_class = uploads.UploadLimitRequest + app.config.update( SECRET_KEY=load_or_create_secret_key(config.secret_key_file), # site_key statt app_key: Cookies unterscheiden keine Ports. Zwei @@ -172,6 +177,9 @@ def create_base_app( SESSION_COOKIE_HTTPONLY=True, SESSION_COOKIE_SECURE=config.security.cookie_secure, SESSION_COOKIE_SAMESITE=config.security.cookie_samesite, + # Nur die aeussere Schranke aus der Umgebung. Die eigentliche Grenze + # steht in der Einstellung und wird pro Anfrage gelesen (uploads.py) -- + # sonst braeuchte jede Aenderung einen Neustart aller Arbeiter. MAX_CONTENT_LENGTH=config.security.max_content_length, JSON_SORT_KEYS=False, TRAP_HTTP_EXCEPTIONS=False, @@ -328,6 +336,16 @@ def create_base_app( # -- Fehlerseiten ------------------------------------------------------ # @app.errorhandler(HTTPException) def _http_error(error: HTTPException): # type: ignore[no-untyped-def] + if error.code == 413: + # "Request Entity Too Large" allein hilft niemandem weiter: die + # Zahl steht in einer Einstellung, und wer hochlaedt, soll sie + # sehen. Vorher stand hier nur der nackte Status. + grenze = uploads.limit_mb() + error.description = ( + f"Diese Datei ist groesser als erlaubt. Zurzeit sind {grenze} MB je Upload " + "eingestellt; aendern kann das ein Administrator unter " + '"Einstellungen -> Webserver und TLS".' + ) if _wants_json(): return { "error": error.name, diff --git a/packages/tesm-core/src/tesm_core/static/tesm_core/css/components.css b/packages/tesm-core/src/tesm_core/static/tesm_core/css/components.css index 958efab..efc3762 100644 --- a/packages/tesm-core/src/tesm_core/static/tesm_core/css/components.css +++ b/packages/tesm-core/src/tesm_core/static/tesm_core/css/components.css @@ -685,7 +685,10 @@ .modal__sub { font-size: var(--fs-xs); color: var(--text-faint); margin-top: 2px; } /* Eine Datei-Vorschau soll die ganze Flaeche des Dialogs nutzen; der uebliche Innenabstand des Dialogkoerpers wuerde sie einrahmen. */ - .modal__body--flush { padding: 0; max-height: none; } + /* Zwei Klassen am selben Element: diese hier muss gewinnen, obwohl + .modal__body weiter unten steht. Mit der doppelten Klasse haengt das + nicht mehr an der Reihenfolge im Stylesheet. */ + .modal__body.modal__body--flush { padding: 0; max-height: none; overflow: visible; } .modal__caption { padding: 0 var(--sp-5) var(--sp-2); color: var(--text-muted); @@ -702,6 +705,74 @@ darunter. --surface-0 gibt es nicht, der Rahmen blieb durchsichtig. */ background: var(--surface-1); } + + /* -- Betrachter: Bild, PDF, Rahmen ------------------------------------- */ + /* Eine feste Hoehe fuer alle drei Anzeigen: sonst springt der Dialog beim + Wechsel zwischen Foto und PDF. */ + .viewer { + position: relative; + display: flex; + flex-direction: column; + /* Was der Dialog uebrig laesst: Kopf, Dateiname und Fusszeile brauchen + zusammen rund 190 Pixel. Ein fester Anteil der Fensterhoehe schob die + Zoomleiste auf kleinen Fenstern unter den Rand. */ + height: min(calc(100dvh - 240px), 760px); + min-height: 220px; + background: var(--surface-1); + } + .viewer > .file-preview { + flex: 1; + height: auto; + } + .viewer__stage { + position: relative; + flex: 1; + /* Ohne min-height waechst die Buehne mit dem Bild, und dessen + max-height:100% haengt dann an sich selbst: ein 4032x3024-Foto stand + 36 Pixel ueber den Rand hinaus. */ + min-height: 0; + display: flex; + align-items: center; + justify-content: center; + overflow: hidden; + /* Die Seite darf nicht mitwandern, wenn im Bild geschoben wird. */ + touch-action: none; + cursor: zoom-in; + } + .viewer__stage--gezogen { + cursor: grab; + } + .viewer__img { + /* Der Ausgangszustand ist "passt in die Flaeche" -- genau das fehlte: + ein Foto vom iPad ist 4032 Pixel breit und wurde im Rahmen 1:1 + gezeigt, also als Ausschnitt aus der Mitte. */ + max-width: 100%; + max-height: 100%; + /* Zoomen soll nicht unscharf wirken, solange nicht vergroessert wird. */ + transform-origin: center center; + will-change: transform; + user-select: none; + -webkit-user-drag: none; + } + /* pdf.js verlangt einen absolut positionierten, scrollenden Behaelter. */ + .viewer__pdf { + position: absolute; + inset: 0; + bottom: var(--sp-10); + overflow: auto; + -webkit-overflow-scrolling: touch; + } + .viewer__bar { + display: flex; + align-items: center; + gap: var(--sp-2); + padding: var(--sp-2) var(--sp-4); + border-top: 1px solid var(--border); + background: var(--surface-2, var(--surface-1)); + } + .viewer__hint { + margin-inline-start: auto; + } /* Der Dateiname ist ein Knopf (er oeffnet die Vorschau), soll aber wie ein Link aussehen -- ein Knopf mit Rahmen mitten in einer Tabellenzeile waere unruhig. */ diff --git a/packages/tesm-core/src/tesm_core/static/tesm_core/js/app.js b/packages/tesm-core/src/tesm_core/static/tesm_core/js/app.js index 419691c..41076f1 100644 --- a/packages/tesm-core/src/tesm_core/static/tesm_core/js/app.js +++ b/packages/tesm-core/src/tesm_core/static/tesm_core/js/app.js @@ -895,6 +895,16 @@ function boot() { initTileFilters(); initLogTails(); unsavedGuard.init(); + initViewer(); +} + +/* Der Dateibetrachter kommt nur auf Seiten mit Vorschau -- und er zieht bei + einem PDF noch pdf.js nach. Beides erst dann, wenn es gebraucht wird. */ +function initViewer() { + if (!document.querySelector("[data-viewer]")) return; + import("./viewer.js") + .then((modul) => modul.initFilePreview()) + .catch((fehler) => console.error("Dateibetrachter", fehler)); } if (document.readyState === "loading") { diff --git a/packages/tesm-core/src/tesm_core/static/tesm_core/js/viewer.js b/packages/tesm-core/src/tesm_core/static/tesm_core/js/viewer.js new file mode 100644 index 0000000..0a568ae --- /dev/null +++ b/packages/tesm-core/src/tesm_core/static/tesm_core/js/viewer.js @@ -0,0 +1,388 @@ +/* ============================================================================= + * Dateivorschau: Bilder zoomen, PDF-Formulare ausfuellen. + * + * Vorher lief jede Vorschau durch einen