netadmin/GDPRScanner
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Bugfix in Scheduler

Browse Source
This commit is contained in:
StyxX65 2026-06-04 14:47:01 +02:00
parent fa6601ffdd
commit 79e589b525
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CLAUDE.md
View File

@ -217,6 +217,7 @@ Allows reviewing results from any past scan session without running a new scan.
- **`scheduler_started` / `scheduler_done` SSE events** — broadcast at start and end of every job (including report-only). `scheduler_done` carries `flagged`, `scanned`, `emailed`, and `job_name`. Do not confuse with `scan_done` (M365) — they are separate event types. - **`scheduler_started` / `scheduler_done` SSE events** — broadcast at start and end of every job (including report-only). `scheduler_done` carries `flagged`, `scanned`, `emailed`, and `job_name`. Do not confuse with `scan_done` (M365) — they are separate event types.
- **UI — job card badge** — `schedRenderJobs()` in `scheduler.js` adds a blue "Report only" (`m365_sched_report_only`) badge to the job name when `j.report_only` is true. - **UI — job card badge** — `schedRenderJobs()` in `scheduler.js` adds a blue "Report only" (`m365_sched_report_only`) badge to the job name when `j.report_only` is true.
- **UI — `schedToggleReportOnly()`** — dims the Profile row (`#schedProfileRow` opacity 0.4), shows/hides `#schedReportOnlyHint`, and forces `#schedAutoEmail` checked. Called from the checkbox `onchange` handler and at the start of `schedAddJob()` / `schedEditJob()`. - **UI — `schedToggleReportOnly()`** — dims the Profile row (`#schedProfileRow` opacity 0.4), shows/hides `#schedReportOnlyHint`, and forces `#schedAutoEmail` checked. Called from the checkbox `onchange` handler and at the start of `schedAddJob()` / `schedEditJob()`.
- **Profile options merge into file sources** — saved file-source dicts (`~/.gdprscanner/file_sources.json`) only carry connection info (`id, label, path, smb_host, smb_user, smb_domain, keychain_key, sftp_*`). Per-scan options live on the profile (`options.cpr_only`, `options.ocr_lang`, `options.scan_photos`, `options.skip_gps_images`, `options.min_cpr_count`, `options.scan_emails`, `options.scan_phones`, `options.max_file_mb`). The scheduler unpacks them with `{**fs, **_fs_extra}` before calling `_m.run_file_scan(fs)` — mirroring what `startScan()` in `scan.js` does in the browser. Do not pass `fs` directly to `run_file_scan()` without merging — the file scan reads these keys with `source.get(...)` and will silently fall back to defaults, ignoring the profile's settings.
## Claude NER — document_scanner.py + app_config.py + routes/app_routes.py ## Claude NER — document_scanner.py + app_config.py + routes/app_routes.py

2
scan_scheduler.py
View File

@ -473,7 +473,7 @@ class ScanScheduler:
logger.info("[scheduler] Profile '%s': sources=%s, users=%d", logger.info("[scheduler] Profile '%s': sources=%s, users=%d",
p.get("name", pid), opts["sources"], len(opts.get("user_ids", []))) p.get("name", pid), opts["sources"], len(opts.get("user_ids", [])))
_m.broadcast("scheduler_debug", { _m.broadcast("scheduler_debug", {
"msg": f"Using profile '{p.get('name',pid)}': sources={opts['sources']}, users={len(opts.get("user_ids",[]))}"}) "msg": f"Using profile '{p.get('name',pid)}': sources={opts['sources']}, users={len(opts.get('user_ids',[]))}"})
return opts return opts
logger.info("[scheduler] Profile '%s' not found — using saved settings", pid) logger.info("[scheduler] Profile '%s' not found — using saved settings", pid)
_m.broadcast("scheduler_debug", {"msg": f"Profile id '{pid}' not found — falling back to saved settings"}) _m.broadcast("scheduler_debug", {"msg": f"Profile id '{pid}' not found — falling back to saved settings"})