docs: TASK.md — Review #5 (M3: Alarm-Mail Deduplizierung fehlt)
This commit is contained in:
parent
33c94e5c20
commit
2363c0dc9b
2 changed files with 18 additions and 0 deletions
18
TASK.md
18
TASK.md
|
|
@ -83,3 +83,21 @@ M1 + M2 fixen → dann **PASS**.
|
|||
- extract_page bei vielen Links (1000): korrekt ✅
|
||||
- compare_snapshots Performance (1000 Seiten): 0.005s ✅
|
||||
- compare_snapshots vergleicht text, nicht text_hash: korrekt ✅
|
||||
|
||||
## Review #5 (Alarm-Mail Deduplizierung)
|
||||
|
||||
**Urteil:** PASS WITH CONCERNS
|
||||
|
||||
### Neues Major
|
||||
|
||||
**M3: Keine Alarm-Mail Deduplizierung**
|
||||
*Datei:* `scanner/alerter.py` (Zeile 1-21), `scanner/__main__.py` (Zeile 740, 815)
|
||||
*Befehl:* `grep -rn "duplicate\|seen_before\|last_alert" scanner/*.py` → **keine Treffer**
|
||||
*Reproduktion:* 3 Scans von jamulix.de heute → 3 identische Alarm-Mails (20:40, 20:43, 20:50)
|
||||
*Problem:* `send_alert()` prüft nur Level ≥ min_level. Gleiche Alarme (neue externe Domains) werden bei jedem Scan erneut gemeldet.
|
||||
*Fix:* Letzten Alarm-Hash speichern und nur bei Änderung senden.
|
||||
|
||||
### Status
|
||||
- Tests: 266/266 grün ✅
|
||||
- M1 + M2: unverändert ✅
|
||||
- M3: NEU — Alarm-Mail Deduplizierung fehlt ❌
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue