summary refs log tree commit diff
diff options
context:
space:
mode:
authorSergey M․ <dstftw@gmail.com>2018-07-21 12:30:18 +0700
committerSergey M․ <dstftw@gmail.com>2018-07-21 12:30:18 +0700
commite9c671d5e86e43785382ae9cb20c8e7676c7c9bf (patch)
tree27cf0a3dbdf3ad03d76fc60e0a774d82a6be6dd0
parentfd62b36680ff7d7bea789ac0031a33fc2d9270ad (diff)
downloadyoutube-dl-e9c671d5e86e43785382ae9cb20c8e7676c7c9bf.tar.gz
youtube-dl-e9c671d5e86e43785382ae9cb20c8e7676c7c9bf.tar.xz
youtube-dl-e9c671d5e86e43785382ae9cb20c8e7676c7c9bf.zip
[utils] Allow JSONP with empty func name (closes #17028)
-rw-r--r--test/test_utils.py4
-rw-r--r--youtube_dl/utils.py2
2 files changed, 5 insertions, 1 deletions
diff --git a/test/test_utils.py b/test/test_utils.py
index e63af0166..de841b1a0 100644
--- a/test/test_utils.py
+++ b/test/test_utils.py
@@ -717,6 +717,10 @@ class TestUtil(unittest.TestCase):
         d = json.loads(stripped)
         self.assertEqual(d, {'status': 'success'})
 
+        stripped = strip_jsonp('({"status": "success"});')
+        d = json.loads(stripped)
+        self.assertEqual(d, {'status': 'success'})
+
     def test_uppercase_escape(self):
         self.assertEqual(uppercase_escape('aä'), 'aä')
         self.assertEqual(uppercase_escape('\\U0001d550'), '𝕐')
diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py
index 8c45166d7..b8700efcb 100644
--- a/youtube_dl/utils.py
+++ b/youtube_dl/utils.py
@@ -2282,7 +2282,7 @@ def parse_age_limit(s):
 def strip_jsonp(code):
     return re.sub(
         r'''(?sx)^
-            (?:window\.)?(?P<func_name>[a-zA-Z0-9_.$]+)
+            (?:window\.)?(?P<func_name>[a-zA-Z0-9_.$]*)
             (?:\s*&&\s*(?P=func_name))?
             \s*\(\s*(?P<callback_data>.*)\);?
             \s*?(?://[^\n]*)*$''',