Repair worker: log debug instead of bare pass on active-server lookup
Ruff S110 (try-except-pass) on the lookup inside `_build_unresolvable_album_folder_error`. Swallowed exception is benign (some test stubs don't expose `get_active_media_server` and we fall back to 'unknown'), but ruff is right that bare pass is a smell. Logger is the existing repair_worker logger, so this matches the same "debug-log on optional-input failure" pattern used in `core/library/path_resolver.py:_collect_base_dirs`.
This commit is contained in:
parent
56ae10693b
commit
7e70bbaaed
1 changed files with 2 additions and 2 deletions
|
|
@ -1949,8 +1949,8 @@ class RepairWorker:
|
|||
active_server = getter() or 'unknown'
|
||||
else:
|
||||
active_server = self._config_manager.get('active_media_server', 'unknown') or 'unknown'
|
||||
except Exception:
|
||||
pass
|
||||
except Exception as e:
|
||||
logger.debug("active media server lookup failed: %s", e)
|
||||
|
||||
lines = [
|
||||
"Could not find any existing track from this album on disk.",
|
||||
|
|
|
|||
Loading…
Reference in a new issue