about summary refs log tree commit diff
path: root/youtube_dl/utils.py
diff options
context:
space:
mode:
authorpukkandan <pukkandan.ytdlp@gmail.com>2021-06-01 18:05:41 +0530
committerdirkf <fieldhouse@gmx.net>2023-02-20 20:27:25 +0000
commit1d3751c3fe50b203d3e2bff71d866c8c500f8288 (patch)
treef114ce1ae17ea24c44f3e3b5e5428822d4040c0d /youtube_dl/utils.py
parent6067451e432fb65d487a8a67bb5cff52efb9ccf4 (diff)
downloadyoutube-dl-1d3751c3fe50b203d3e2bff71d866c8c500f8288.tar.gz
youtube-dl-1d3751c3fe50b203d3e2bff71d866c8c500f8288.tar.xz
youtube-dl-1d3751c3fe50b203d3e2bff71d866c8c500f8288.zip
Escape URLs in `sanitized_Request`, not `sanitize_url` d2558234cf5dd12d6896eed5427b7dcdb3ab7b5a added escaping of URLs while sanitizing. However, `sanitize_url` may not always receive an actual URL. Eg: When using `youtube-dl "search query" --default-search ytsearch`, `search query` gets escaped to `search%20query` before being prefixed with `ytsearch:` which is not the intended behavior. So the escaping is moved to `sanitized_Request` instead.
Diffstat (limited to 'youtube_dl/utils.py')
-rw-r--r--youtube_dl/utils.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py
index 4edbfa27b..761edcd49 100644
--- a/youtube_dl/utils.py
+++ b/youtube_dl/utils.py
@@ -2176,11 +2176,11 @@ def sanitize_url(url):
     for mistake, fixup in COMMON_TYPOS:
         if re.match(mistake, url):
             return re.sub(mistake, fixup, url)
-    return escape_url(url)
+    return url
 
 
 def sanitized_Request(url, *args, **kwargs):
-    return compat_urllib_request.Request(sanitize_url(url), *args, **kwargs)
+    return compat_urllib_request.Request(escape_url(sanitize_url(url)), *args, **kwargs)
 
 
 def expand_path(s):