warn when invalid numeric INI values silently fall back to defaults #19

Open
opened 2026-03-16 06:36:40 -04:00 by david · 0 comments
Owner

Numeric config fields like MAX_VIDEO_HEIGHT, CHANNEL_CACHE_EXPIRES_HOURS, CURL_TIMEOUT, and RESCAN_TIMEOUT silently use defaults when parsing fails (e.g. MAX_VIDEO_HEIGHT=720p). No warning is logged, so users have no idea their setting was ignored.

Fix: log a [WARN] when a numeric parse fails and the default is used.

Numeric config fields like MAX_VIDEO_HEIGHT, CHANNEL_CACHE_EXPIRES_HOURS, CURL_TIMEOUT, and RESCAN_TIMEOUT silently use defaults when parsing fails (e.g. `MAX_VIDEO_HEIGHT=720p`). No warning is logged, so users have no idea their setting was ignored. **Fix:** log a `[WARN]` when a numeric parse fails and the default is used.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
highseas/yt-sonarr-downloader#19
No description provided.