Skip to content

Commit

Permalink
[gamespot] lower the preference of http formats(#14652)
Browse files Browse the repository at this point in the history
  • Loading branch information
remitamine committed Nov 13, 2017
1 parent 5fc12b9 commit d4e31b7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion youtube_dl/extractor/gamespot.py
Expand Up @@ -109,7 +109,7 @@ def _real_extract(self, url):
if onceux_url:
formats.extend(self._extract_once_formats(re.sub(
r'https?://[^/]+', 'http://once.unicornmedia.com', onceux_url),
skip_http_formats=True))
http_formats_preference=-1))

if not formats:
for quality in ['sd', 'hd']:
Expand Down
5 changes: 3 additions & 2 deletions youtube_dl/extractor/once.py
Expand Up @@ -11,7 +11,7 @@ class OnceIE(InfoExtractor):
ADAPTIVE_URL_TEMPLATE = 'http://once.unicornmedia.com/now/master/playlist/%s/%s/%s/content.m3u8'
PROGRESSIVE_URL_TEMPLATE = 'http://once.unicornmedia.com/now/media/progressive/%s/%s/%s/%s/content.mp4'

def _extract_once_formats(self, url, skip_http_formats=False):
def _extract_once_formats(self, url, http_formats_preference=None):
domain_id, application_id, media_item_id = re.match(
OnceIE._VALID_URL, url).groups()
formats = self._extract_m3u8_formats(
Expand All @@ -27,14 +27,15 @@ def _extract_once_formats(self, url, skip_http_formats=False):
rendition_id = self._search_regex(
r'/now/media/playlist/[^/]+/[^/]+/([^/]+)',
adaptive_format['url'], 'redition id', default=None)
if rendition_id and not skip_http_formats:
if rendition_id:
progressive_format = adaptive_format.copy()
progressive_format.update({
'url': self.PROGRESSIVE_URL_TEMPLATE % (
domain_id, application_id, rendition_id, media_item_id),
'format_id': adaptive_format['format_id'].replace(
'hls', 'http'),
'protocol': 'http',
'preference': http_formats_preference,
})
progressive_formats.append(progressive_format)
self._check_formats(progressive_formats, media_item_id)
Expand Down

0 comments on commit d4e31b7

Please sign in to comment.