Browse Source

[vimeo] Clarify new react+flux website fallback

Sergey M․ 10 years ago
parent
commit
dd8417526b
1 changed files with 5 additions and 2 deletions
  1. 5 2
      youtube_dl/extractor/vimeo.py

+ 5 - 2
youtube_dl/extractor/vimeo.py

@@ -289,11 +289,14 @@ class VimeoIE(VimeoBaseInfoExtractor):
                     r' data-config-url="(.+?)"', webpage,
                     r' data-config-url="(.+?)"', webpage,
                     'config URL', default=None)
                     'config URL', default=None)
                 if not config_url:
                 if not config_url:
-                    # New react-based page
+                    # Sometimes new react-based page is served instead of old one that require
+                    # different config URL extraction approach (see
+                    # https://github.com/rg3/youtube-dl/pull/7209)
                     vimeo_clip_page_config = self._search_regex(
                     vimeo_clip_page_config = self._search_regex(
                         r'vimeo\.clip_page_config\s*=\s*({.+?});', webpage,
                         r'vimeo\.clip_page_config\s*=\s*({.+?});', webpage,
                         'vimeo clip page config')
                         'vimeo clip page config')
-                    config_url = self._parse_json(vimeo_clip_page_config, video_id)['player']['config_url']
+                    config_url = self._parse_json(
+                        vimeo_clip_page_config, video_id)['player']['config_url']
                 config_json = self._download_webpage(config_url, video_id)
                 config_json = self._download_webpage(config_url, video_id)
                 config = json.loads(config_json)
                 config = json.loads(config_json)
             except RegexNotFoundError:
             except RegexNotFoundError: