Merge pull request #6 from bbtufty/package-name

Sanitize JDownloader package name

CHANGES.rst

@@ -1,3 +1,8 @@
+0.2 (Unreleased)
+================
+
+- Ensure package name is valid for JDownloader [#6]
+
 0.1 (2024-10-19)
 ================
 

nxbrew_dl/nxbrew_dl/nxbrew.py

@@ -2,6 +2,7 @@
 import os
 import time
 import shutil
+from pathvalidate import sanitize_filename
 
 import myjdapi
 
@@ -278,10 +279,13 @@ def download_game(
                     self.logger.info(f"Downloading {dl_key}: {dl_info['full_name']}")
                     out_dir = os.path.join(self.user_config["download_dir"], dl_dir)
 
+                    # Sanitize the package name so we're safe here
+                    package_name = sanitize_filename(name)
+
                     self.run_jdownloader(
                         dl_dict=dl_info,
                         out_dir=out_dir,
-                        package_name=name,
+                        package_name=package_name,
                     )
 
                     # Update and save out cache

pyproject.toml

@@ -37,6 +37,7 @@ dependencies = [
     "discordwebhook>=1.0.3",
     "lxml>=0.5.3",
     "myjdapi>=1.1.7",
+    "pathvalidate>=3.2.1",
     "PySide6>=6.7.3",
     "requests>=2.32.3",
     "PyYAML>=6.0.2",