diff --git a/test/test_utils.py b/test/test_utils.py index f1a748dde..e8eb8b216 100644 --- a/test/test_utils.py +++ b/test/test_utils.py @@ -65,6 +65,7 @@ from youtube_dl.utils import ( sanitize_filename, sanitize_path, sanitize_url, + sanitized_Request, expand_path, prepend_extension, replace_extension, @@ -236,6 +237,10 @@ class TestUtil(unittest.TestCase): self.assertEqual(sanitize_url('httpss://foo.bar'), 'https://foo.bar') self.assertEqual(sanitize_url('rmtps://foo.bar'), 'rtmps://foo.bar') self.assertEqual(sanitize_url('https://foo.bar'), 'https://foo.bar') + self.assertEqual(sanitize_url('ytsearch:search query'), 'ytsearch:search query') + + def test_sanitized_Request(self): + self.assertEqual(sanitized_Request('https://foo.bar/foo bar').get_full_url(), 'https://foo.bar/foo%20bar') def test_expand_path(self): def env(var): diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py index fea38ed32..49494b7b7 100644 --- a/youtube_dl/utils.py +++ b/youtube_dl/utils.py @@ -2169,11 +2169,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):