Zeitstempel und lesbares Format fürs Ausführen-Log #5

Merged

DasAoD merged 2 commits from feature/log-timestamps-readable into main

2026-08-22 19:22:44 +02:00

DasAoD commented

2026-08-22 19:12:28 +02:00

Owner

Was

Das Log unter „▶️ Ausführen" war bisher 1:1 die rohe imapfilter-Ausgabe — ganz ohne Zeitstempel und mit sehr technischen Zeilen wie:

1 messages moved from m032bb7c@w010ea06.kasserver.com/INBOX to m032bb7c@w010ea06.kasserver.com/#fritz!box.

Jetzt:

Zeitstempel — ein [YYYY-MM-DD HH:MM:SS] pro Lauf (Dispatcher wie manueller Klick auf „imapfilter starten"), damit erkennbar ist, wann zuletzt Mails verschoben wurden.
Lesbare Kurzform in der Anzeige: N messages moved from user@host/A to user@host/B. → 📬 N Mail(s) verschoben: A → B. Warnungen/Fehler von imapfilter selbst werden mit ⚠️ markiert statt unauffällig dazwischenzustehen.
Manuelle Läufe werden jetzt persistiert — vorher landete die Ausgabe eines manuellen „imapfilter starten"-Klicks nur kurz transient im Browser und war beim nächsten Neuladen weg. Jetzt landet sie (mit (manuell)-Markierung) in derselben Logdatei wie die Cron-Läufe.
Die gespeicherte Rohdatei bleibt unverändert — nur die Ansicht im Browser wird aufbereitet, damit bei Bedarf (Fehlersuche) trotzdem der Originaltext einsehbar ist (z.B. per SSH).

Wie

cron/dispatcher.php: Ausgabe wird jetzt über exec()s $output-Array eingesammelt statt per Shell-Redirect direkt in die Logdatei geschrieben, dann mit einem Zeitstempel pro Lauf per PHP an die Datei angehängt.
public/api/run.php: manuelle Läufe schreiben jetzt genauso ins persistente Log.
public/assets/app.js: loadLog() formatiert jede Zeile fürs Anzeigen um (formatLogLine()), lässt die Rohdatei aber unangetastet.

Review

code-reviewer bestätigt: keine Verhaltensänderung beim Exit-Code ($code kam vorher wie nachher vom exec()-Kommando selbst, die Shell-Umleitung hatte darauf nie Einfluss), sauberes Array-Reset pro Nutzer in der Dispatcher-Schleife, Regex robust für die realistischen Fälle, kein XSS (nur textContent, nie innerHTML). Zwei kleine Punkte behoben: leere Log-Zeilen erzeugten einen inhaltsleeren Zeitstempel-Eintrag (jetzt rausgefiltert), und manuelle Lauf-Zeilen hatten ein doppeltes Emoji (🖱️ + 📬) — vereinfacht zu einem Textzusatz „· manueller Lauf".

🤖 Generated with Claude Code

## Was Das Log unter „▶️ Ausführen" war bisher 1:1 die rohe `imapfilter`-Ausgabe — ganz ohne Zeitstempel und mit sehr technischen Zeilen wie: ``` 1 messages moved from m032bb7c@w010ea06.kasserver.com/INBOX to m032bb7c@w010ea06.kasserver.com/#fritz!box. ``` Jetzt: - **Zeitstempel** — ein `[YYYY-MM-DD HH:MM:SS]` pro Lauf (Dispatcher wie manueller Klick auf „imapfilter starten"), damit erkennbar ist, wann zuletzt Mails verschoben wurden. - **Lesbare Kurzform** in der Anzeige: `N messages moved from user@host/A to user@host/B.` → `📬 N Mail(s) verschoben: A → B`. Warnungen/Fehler von imapfilter selbst werden mit ⚠️ markiert statt unauffällig dazwischenzustehen. - **Manuelle Läufe werden jetzt persistiert** — vorher landete die Ausgabe eines manuellen „imapfilter starten"-Klicks nur kurz transient im Browser und war beim nächsten Neuladen weg. Jetzt landet sie (mit `(manuell)`-Markierung) in derselben Logdatei wie die Cron-Läufe. - Die **gespeicherte Rohdatei bleibt unverändert** — nur die Ansicht im Browser wird aufbereitet, damit bei Bedarf (Fehlersuche) trotzdem der Originaltext einsehbar ist (z.B. per SSH). ## Wie - `cron/dispatcher.php`: Ausgabe wird jetzt über `exec()`s `$output`-Array eingesammelt statt per Shell-Redirect direkt in die Logdatei geschrieben, dann mit einem Zeitstempel pro Lauf per PHP an die Datei angehängt. - `public/api/run.php`: manuelle Läufe schreiben jetzt genauso ins persistente Log. - `public/assets/app.js`: `loadLog()` formatiert jede Zeile fürs Anzeigen um (`formatLogLine()`), lässt die Rohdatei aber unangetastet. ## Review `code-reviewer` bestätigt: keine Verhaltensänderung beim Exit-Code (`$code` kam vorher wie nachher vom `exec()`-Kommando selbst, die Shell-Umleitung hatte darauf nie Einfluss), sauberes Array-Reset pro Nutzer in der Dispatcher-Schleife, Regex robust für die realistischen Fälle, kein XSS (nur `textContent`, nie `innerHTML`). Zwei kleine Punkte behoben: leere Log-Zeilen erzeugten einen inhaltsleeren Zeitstempel-Eintrag (jetzt rausgefiltert), und manuelle Lauf-Zeilen hatten ein doppeltes Emoji (🖱️ + 📬) — vereinfacht zu einem Textzusatz „· manueller Lauf". 🤖 Generated with [Claude Code](https://claude.com/claude-code)

DasAoD added 2 commits

2026-08-22 19:12:28 +02:00

cron/dispatcher.php, public/api/run.php, public/assets/app.js: add timestamps and readable formatting to the run log 45f0baa1be

- imapfilter's own output has no timestamps at all ("N messages moved from
  user@host/A to user@host/B.") and both the cron dispatcher and manual
  runs wrote it via shell redirect with no time reference
- cron/dispatcher.php: capture imapfilter's output via exec()'s $output
  array instead of a shell "2>&1 >> logfile" redirect, then write it to
  the log with one timestamp per run — also means $code now reflects
  imapfilter's real exit status directly (no behavior change there, just
  no longer routed through a shell redirect)
- public/api/run.php: manual "imapfilter starten" runs now get persisted
  to the same log file the same way (tagged "(manuell)"), instead of only
  flashing transiently in the browser and never showing up on reload
- public/assets/app.js: loadLog() reformats each line for display — timestamp
  kept, "N messages moved from user@host/A to user@host/B." collapsed to
  "N Mail(s) verschoben: A → B", imapfilter's own diagnostic lines prefixed
  with ⚠️ instead of looking identical to a normal move. The stored log file
  itself is untouched, only the rendered view is prettified.

Co-authored-by: Claude <claude@anthropic.com>

cron/dispatcher.php, public/api/run.php, public/assets/app.js: address review feedback a4df9c7c77

- Filter out blank lines from imapfilter's output before writing to the
  log — previously a run with an empty output line still produced a
  bare "[timestamp]" entry with no content
- Drop the redundant 🖱️ emoji on manual-run lines (already carries the
  move emoji or a warning marker); just append "· manueller Lauf" instead

Co-authored-by: Claude <claude@anthropic.com>

DasAoD merged commit 8eb0cc294e into main

2026-08-22 19:22:44 +02:00

DasAoD referenced this pull request from a commit

2026-08-22 19:22:45 +02:00

Merge pull request 'Zeitstempel und lesbares Format fürs Ausführen-Log' (#5) from feature/log-timestamps-readable into main