IDLE wakeup now logs every processed UID — document and consider gating #30

New issue

Closed

opened 2026-05-12 09:34:45 +02:00 by heiko · 2 comments

heiko commented

2026-05-12 09:34:45 +02:00

Owner

Found during review of #24.

With ReportProcessed: true in the IDLE path, every UID processed after an IDLE wakeup is now logged (not just matches). On a busy mailbox this can substantially increase log volume.

Two small follow-ups:

Document this under the scan=idle description in doc/admin.md so operators expecting "only matches logged" aren't surprised.
Consider whether ReportProcessed should be gated by --verbose or a dedicated flag, instead of being unconditional on the IDLE path. The startup/initial paths deliberately stay quiet for unmatched messages; IDLE diverges from that convention.

Refs: PR #24

Found during review of #24. With `ReportProcessed: true` in the IDLE path, every UID processed after an IDLE wakeup is now logged (not just matches). On a busy mailbox this can substantially increase log volume. Two small follow-ups: 1. Document this under the `scan=idle` description in `doc/admin.md` so operators expecting "only matches logged" aren't surprised. 2. Consider whether `ReportProcessed` should be gated by `--verbose` or a dedicated flag, instead of being unconditional on the IDLE path. The startup/initial paths deliberately stay quiet for unmatched messages; IDLE diverges from that convention. Refs: PR #24

heiko added the

finding

label

2026-05-12 09:34:45 +02:00

heiko commented

2026-05-13 14:03:46 +02:00

Author

Owner

Related commit: 2305c33. The idle wakeup path still reports every processed UID with scan=idle, so this behavior is still present.

Related commit: 2305c33. The idle wakeup path still reports every processed UID with scan=idle, so this behavior is still present.

heiko commented

2026-05-15 23:24:59 +02:00

Author

Owner

Fixed in ef3461b: ReportProcessed in the IDLE path is now gated on opts.verbose (the --verbose flag). Without --verbose, only matches are logged — consistent with startup/initial. doc/admin.md updated to document the behaviour.

Fixed in ef3461b: `ReportProcessed` in the IDLE path is now gated on `opts.verbose` (the `--verbose` flag). Without `--verbose`, only matches are logged — consistent with startup/initial. `doc/admin.md` updated to document the behaviour.

heiko closed this issue

2026-05-15 23:25:00 +02:00

heiko referenced this issue from a commit

2026-05-15 23:40:49 +02:00

idle: gate per-UID logging behind --verbose (closes #30)

heiko referenced this issue

2026-05-15 23:41:10 +02:00

fix: address PR #24 review findings (#26, #27, #29, #30) #33

heiko referenced this issue from a commit

2026-05-15 23:48:13 +02:00

idle: gate per-UID logging behind --verbose (closes #30)