[gamespot] Add support for videos that don't use 'f4m_stream' (fixes #5707)

This commit is contained in:
Jaime Marquínez Ferrándiz 2015-05-15 18:42:59 +02:00
parent 62c95fd5fc
commit 34fe5a94ba

View file

@ -15,7 +15,7 @@
class GameSpotIE(InfoExtractor): class GameSpotIE(InfoExtractor):
_VALID_URL = r'(?:http://)?(?:www\.)?gamespot\.com/.*-(?P<id>\d+)/?' _VALID_URL = r'(?:http://)?(?:www\.)?gamespot\.com/.*-(?P<id>\d+)/?'
_TEST = { _TESTS = [{
'url': 'http://www.gamespot.com/videos/arma-3-community-guide-sitrep-i/2300-6410818/', 'url': 'http://www.gamespot.com/videos/arma-3-community-guide-sitrep-i/2300-6410818/',
'md5': 'b2a30deaa8654fcccd43713a6b6a4825', 'md5': 'b2a30deaa8654fcccd43713a6b6a4825',
'info_dict': { 'info_dict': {
@ -23,8 +23,16 @@ class GameSpotIE(InfoExtractor):
'ext': 'mp4', 'ext': 'mp4',
'title': 'Arma 3 - Community Guide: SITREP I', 'title': 'Arma 3 - Community Guide: SITREP I',
'description': 'Check out this video where some of the basics of Arma 3 is explained.', 'description': 'Check out this video where some of the basics of Arma 3 is explained.',
} },
} }, {
'url': 'http://www.gamespot.com/videos/the-witcher-3-wild-hunt-xbox-one-now-playing/2300-6424837/',
'info_dict': {
'id': 'gs-2300-6424837',
'ext': 'flv',
'title': 'The Witcher 3: Wild Hunt [Xbox ONE] - Now Playing',
'description': 'Join us as we take a look at the early hours of The Witcher 3: Wild Hunt and more.',
},
}]
def _real_extract(self, url): def _real_extract(self, url):
page_id = self._match_id(url) page_id = self._match_id(url)
@ -32,10 +40,13 @@ def _real_extract(self, url):
data_video_json = self._search_regex( data_video_json = self._search_regex(
r'data-video=["\'](.*?)["\']', webpage, 'data video') r'data-video=["\'](.*?)["\']', webpage, 'data video')
data_video = json.loads(unescapeHTML(data_video_json)) data_video = json.loads(unescapeHTML(data_video_json))
streams = data_video['videoStreams']
formats = []
f4m_url = streams.get('f4m_stream')
if f4m_url is not None:
# Transform the manifest url to a link to the mp4 files # Transform the manifest url to a link to the mp4 files
# they are used in mobile devices. # they are used in mobile devices.
f4m_url = data_video['videoStreams']['f4m_stream']
f4m_path = compat_urlparse.urlparse(f4m_url).path f4m_path = compat_urlparse.urlparse(f4m_url).path
QUALITIES_RE = r'((,\d+)+,?)' QUALITIES_RE = r'((,\d+)+,?)'
qualities = self._search_regex(QUALITIES_RE, f4m_path, 'qualities').strip(',').split(',') qualities = self._search_regex(QUALITIES_RE, f4m_path, 'qualities').strip(',').split(',')
@ -44,13 +55,22 @@ def _real_extract(self, url):
http_template = http_template.replace('.csmil/manifest.f4m', '') http_template = http_template.replace('.csmil/manifest.f4m', '')
http_template = compat_urlparse.urljoin( http_template = compat_urlparse.urljoin(
'http://video.gamespotcdn.com/', http_template) 'http://video.gamespotcdn.com/', http_template)
formats = []
for q in qualities: for q in qualities:
formats.append({ formats.append({
'url': http_template % q, 'url': http_template % q,
'ext': 'mp4', 'ext': 'mp4',
'format_id': q, 'format_id': q,
}) })
else:
for quality in ['sd', 'hd']:
# It's actually a link to a flv file
flv_url = streams.get('f4m_{0}'.format(quality))
if flv_url is not None:
formats.append({
'url': flv_url,
'ext': 'flv',
'format_id': quality,
})
return { return {
'id': data_video['guid'], 'id': data_video['guid'],