fix: sprechende Dateinamen fuer alle Exporte
Im Downloads-Ordner lagen "certain-finance-export_....json" und "certain-finance-backup_....json" nebeneinander, ohne dass man sah, welche Datei sich zurueckladen laesst. Alle Downloads heissen jetzt certain-finance_<art>_<datum>_<uhrzeit>.<endung>, wobei die Art es sagt: vollbackup, vollbackup-system, datenuebersicht, buchungen, datenbank. Das persoenliche Vollbackup traegt zusaetzlich den Benutzernamen. Den Namen vergibt jetzt allein der Server im Content-Disposition-Kopf; das Frontend liest ihn von dort. Vorher erfanden beide Seiten ihn getrennt und kamen auf unterschiedliche Ergebnisse (Bindestrich gegen Unterstrich, UTC gegen Ortszeit). Zeitstempel ist Ortszeit: wer abends um 20 Uhr exportiert, soll keine Datei von 18 Uhr finden. Passend dazu heisst der Knopf "Alle Daten (JSON)" nun "Datenuebersicht (JSON)", und der Hinweis darueber sagt ausdruecklich, dass sich diese Dateien nicht zurueckladen lassen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
7fc8763d57
commit
e503baa8a6
@@ -4,6 +4,7 @@ das wiederherstellbare Voll-Backup als JSON (siehe ``services/backup_data``)."""
|
||||
import csv
|
||||
import io
|
||||
import os
|
||||
import re
|
||||
import sqlite3
|
||||
import tempfile
|
||||
from datetime import datetime, timezone
|
||||
@@ -38,7 +39,27 @@ MAX_BACKUP_BYTES = 64 * 1024 * 1024
|
||||
|
||||
|
||||
def _ts() -> str:
|
||||
return datetime.now(timezone.utc).strftime("%Y%m%d-%H%M%S")
|
||||
"""Ortszeit als ``YYYY-MM-DD_HHMM`` – Dateinamen sollen lesbar sein.
|
||||
|
||||
Bewusst nicht UTC: wer abends um 20 Uhr exportiert, soll im Downloads-Ordner
|
||||
keine Datei von 18 Uhr finden."""
|
||||
return datetime.now().astimezone().strftime("%Y-%m-%d_%H%M")
|
||||
|
||||
|
||||
def _dateiname(art: str, endung: str) -> str:
|
||||
"""Einheitlicher, sprechender Dateiname für alle Downloads.
|
||||
|
||||
``art`` muss erkennen lassen, worum es geht – im Downloads-Ordner liegen die
|
||||
Dateien sonst nebeneinander und man sieht ihnen nicht an, welche sich
|
||||
zurückladen lässt (``vollbackup``) und welche nur zum Ansehen taugt
|
||||
(``datenuebersicht``)."""
|
||||
return f"certain-finance_{art}_{_ts()}.{endung}"
|
||||
|
||||
|
||||
def _namensteil(text: str) -> str:
|
||||
"""Beliebigen Text auf etwas eindampfen, das in einem Dateinamen taugt."""
|
||||
sauber = re.sub(r"[^A-Za-z0-9]+", "-", text).strip("-").lower()
|
||||
return sauber[:32] or "unbekannt"
|
||||
|
||||
|
||||
def _sqlite_path() -> str | None:
|
||||
@@ -94,7 +115,7 @@ def backup_db(_: User = Depends(require_admin)):
|
||||
return FileResponse(
|
||||
tmp.name,
|
||||
media_type="application/octet-stream",
|
||||
filename=f"certain-finance-{_ts()}.db",
|
||||
filename=_dateiname("datenbank", "db"),
|
||||
background=BackgroundTask(os.remove, tmp.name),
|
||||
)
|
||||
|
||||
@@ -136,7 +157,7 @@ def export_transactions_csv(db: Session = Depends(get_db), user: User = Depends(
|
||||
return Response(
|
||||
content=buf.getvalue().encode("utf-8-sig"),
|
||||
media_type="text/csv",
|
||||
headers={"Content-Disposition": f'attachment; filename="transaktionen-{_ts()}.csv"'},
|
||||
headers={"Content-Disposition": f'attachment; filename="{_dateiname("buchungen", "csv")}"'},
|
||||
)
|
||||
|
||||
|
||||
@@ -165,7 +186,7 @@ def export_transactions_json(db: Session = Depends(get_db), user: User = Depends
|
||||
return Response(
|
||||
content=_json_bytes(data),
|
||||
media_type="application/json",
|
||||
headers={"Content-Disposition": f'attachment; filename="transaktionen-{_ts()}.json"'},
|
||||
headers={"Content-Disposition": f'attachment; filename="{_dateiname("buchungen", "json")}"'},
|
||||
)
|
||||
|
||||
|
||||
@@ -229,32 +250,42 @@ def export_all_json(db: Session = Depends(get_db), user: User = Depends(get_curr
|
||||
return Response(
|
||||
content=_json_bytes(data),
|
||||
media_type="application/json",
|
||||
headers={"Content-Disposition": f'attachment; filename="certain-finance-export-{_ts()}.json"'},
|
||||
# „datenuebersicht", nicht „backup": diese Datei lässt sich bewusst
|
||||
# nicht zurückladen (siehe services/backup_data.parse).
|
||||
headers={
|
||||
"Content-Disposition": f'attachment; filename="{_dateiname("datenuebersicht", "json")}"'
|
||||
},
|
||||
)
|
||||
|
||||
|
||||
# --- Voll-Backup (wiederherstellbar) ---------------------------------------
|
||||
|
||||
|
||||
def _download(obj: dict, name: str) -> Response:
|
||||
def _download(obj: dict, dateiname: str) -> Response:
|
||||
return Response(
|
||||
content=backup_data.to_bytes(obj),
|
||||
media_type="application/json",
|
||||
headers={"Content-Disposition": f'attachment; filename="{name}-{_ts()}.json"'},
|
||||
headers={"Content-Disposition": f'attachment; filename="{dateiname}"'},
|
||||
)
|
||||
|
||||
|
||||
@router.get("/export/backup.json")
|
||||
def export_backup(db: Session = Depends(get_db), user: User = Depends(get_current_user)):
|
||||
"""Alle eigenen Daten – vollständig und wieder einspielbar."""
|
||||
return _download(backup_data.export_user(db, user), "certain-finance-backup")
|
||||
"""Alle eigenen Daten – vollständig und wieder einspielbar.
|
||||
|
||||
Der Benutzername steht im Dateinamen: bei mehreren Konten liegen sonst
|
||||
gleichnamige Backups im selben Ordner."""
|
||||
return _download(
|
||||
backup_data.export_user(db, user),
|
||||
_dateiname(f"vollbackup_{_namensteil(user.username)}", "json"),
|
||||
)
|
||||
|
||||
|
||||
@router.get("/export/system-backup.json")
|
||||
def export_system_backup(db: Session = Depends(get_db), _: User = Depends(require_admin)):
|
||||
"""Die ganze Instanz inklusive Benutzerkonten. Enthält Passwort-Hashes und
|
||||
TOTP-Secrets – die Datei ist wie ein Passwort zu behandeln."""
|
||||
return _download(backup_data.export_system(db), "certain-finance-system-backup")
|
||||
return _download(backup_data.export_system(db), _dateiname("vollbackup-system", "json"))
|
||||
|
||||
|
||||
async def _read_upload(file: UploadFile) -> bytes:
|
||||
|
||||
Reference in New Issue
Block a user