ccMixter: Handle spaces in URLs (#3907)

WordPress · Mar 13, 2024 · 4176f6e · 4176f6e
1 parent d80b590
commit 4176f6e
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/catalog/dags/providers/provider_api_scripts/cc_mixter.py b/catalog/dags/providers/provider_api_scripts/cc_mixter.py
@@ -193,7 +193,7 @@ def _get_audio_files(
 
         files = [
             {
-                "url": file["download_url"],
+                "url": file["download_url"].replace(" ", "%20"),
                 "filesize": file["file_rawsize"],
                 "filetype": file["file_format_info"]["default-ext"],
                 "sample_rate": self._get_sample_rate(

diff --git a/...tests/dags/providers/provider_api_scripts/resources/cc_mixter/expected_single_record.json b/...tests/dags/providers/provider_api_scripts/resources/cc_mixter/expected_single_record.json
@@ -30,14 +30,14 @@
     "VBR",
     "instrumental"
   ],
-  "url": "https://ccmixter.org/content/testuser/testuser_-_Test_Music.mp3",
+  "url": "https://ccmixter.org/content/testuser/testuser%20-%20Test_Music.mp3",
   "sample_rate": 44100,
   "filetype": "mp3",
   "filesize": 11150278,
   "duration": 346000,
   "alt_files": [
     {
-      "url": "https://ccmixter.org/content/testuser/testuser_-_Test_Music.flac",
+      "url": "https://ccmixter.org/content/testuser/testuser%20-%20Test_Music.flac",
       "sample_rate": 44000,
       "filetype": "flac",
       "filesize": 111502780,

diff --git a/catalog/tests/dags/providers/provider_api_scripts/resources/cc_mixter/single_item.json b/catalog/tests/dags/providers/provider_api_scripts/resources/cc_mixter/single_item.json
@@ -46,7 +46,7 @@
       "file_order": 0,
       "file_is_remote": 0,
       "file_num_download": 0,
-      "download_url": "https://ccmixter.org/content/testuser/testuser_-_Test_Music.zip",
+      "download_url": "https://ccmixter.org/content/testuser/testuser - Test_Music.zip",
       "local_path": "/var/www/ccmixter/content/testuser/testuser_-_Test_Music.zip",
       "file_rawsize": 11150278
     },
@@ -74,7 +74,7 @@
       "file_order": 0,
       "file_is_remote": 0,
       "file_num_download": 0,
-      "download_url": "https://ccmixter.org/content/testuser/testuser_-_Test_Music.flac",
+      "download_url": "https://ccmixter.org/content/testuser/testuser - Test_Music.flac",
       "local_path": "/var/www/ccmixter/content/testuser/testuser_-_Test_Music.flac",
       "file_rawsize": 111502780
     },
@@ -100,7 +100,7 @@
       "file_order": 0,
       "file_is_remote": 0,
       "file_num_download": 0,
-      "download_url": "https://ccmixter.org/content/testuser/testuser_-_Test_Music.mp3",
+      "download_url": "https://ccmixter.org/content/testuser/testuser - Test_Music.mp3",
       "local_path": "/var/www/ccmixter/content/testuser/testuser_-_Test_Music.mp3",
       "file_rawsize": 11150278
     }