✨ Exception logging on filename sanitization and download watcher.

exislow · Apr 22, 2024 · 840f23d · 840f23d
1 parent c3519c5
commit 840f23d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/tidal_dl_ng/gui.py b/tidal_dl_ng/gui.py
@@ -810,7 +810,8 @@ def watcher_queue_download(self) -> None:
                         self.s_queue_download_item_finished.emit(item)
                     elif result == QueueDownloadStatus.Skipped:
                         self.s_queue_download_item_skipped.emit(item)
-                except:
+                except e:
+                    logger_gui.error(e)
                     self.s_queue_download_item_failed.emit(item)
             else:
                 time.sleep(2)

diff --git a/tidal_dl_ng/helper/path.py b/tidal_dl_ng/helper/path.py
@@ -207,7 +207,6 @@ def path_file_sanitize(path_file: str, adapt: bool = False, uniquify: bool = Fal
             filename_sanitized = filename_sanitized[: -len(file_suffix)] + file_suffix
     except ValidationError as e:
         # TODO: Implement proper exception handling and logging.
-        # print(e)
         # Hacky stuff, since the sanitizing function does not shorten the filename somehow (bug?)
         # TODO: Remove after pathvalidate update.
         # If filename too long
@@ -218,6 +217,8 @@ def path_file_sanitize(path_file: str, adapt: bool = False, uniquify: bool = Fal
                 + FILENAME_SANITIZE_PLACEHOLDER
                 + file_extension
             )
+        else:
+            print(e)
 
     # Join path and filename
     result: str = os.path.join(pathname_sanitized, filename_sanitized)