[common] Fix non-bootstrapped support in f4m

This commit is contained in:
Yen Chi Hsuan 2016-05-27 00:03:03 +08:00
parent 293c255688
commit 448bb5f333
No known key found for this signature in database
GPG key ID: 3FDDD575826C5C30
2 changed files with 14 additions and 7 deletions

View file

@ -987,7 +987,7 @@ class InfoExtractor(object):
def _extract_f4m_formats(self, manifest_url, video_id, preference=None, f4m_id=None, def _extract_f4m_formats(self, manifest_url, video_id, preference=None, f4m_id=None,
transform_source=lambda s: fix_xml_ampersands(s).strip(), transform_source=lambda s: fix_xml_ampersands(s).strip(),
fatal=True, assume_f4mv2=False, m3u8_id=None): fatal=True, m3u8_id=None):
manifest = self._download_xml( manifest = self._download_xml(
manifest_url, video_id, 'Downloading f4m manifest', manifest_url, video_id, 'Downloading f4m manifest',
'Unable to download f4m manifest', 'Unable to download f4m manifest',
@ -1001,12 +1001,11 @@ class InfoExtractor(object):
return self._parse_f4m_formats( return self._parse_f4m_formats(
manifest, manifest_url, video_id, preference=preference, f4m_id=f4m_id, manifest, manifest_url, video_id, preference=preference, f4m_id=f4m_id,
transform_source=transform_source, fatal=fatal, assume_f4mv2=assume_f4mv2, transform_source=transform_source, fatal=fatal, m3u8_id=m3u8_id)
m3u8_id=m3u8_id)
def _parse_f4m_formats(self, manifest, manifest_url, video_id, preference=None, f4m_id=None, def _parse_f4m_formats(self, manifest, manifest_url, video_id, preference=None, f4m_id=None,
transform_source=lambda s: fix_xml_ampersands(s).strip(), transform_source=lambda s: fix_xml_ampersands(s).strip(),
fatal=True, assume_f4mv2=False, m3u8_id=None): fatal=True, m3u8_id=None):
# currently youtube-dl cannot decode the playerVerificationChallenge as Akamai uses Adobe Alchemy # currently youtube-dl cannot decode the playerVerificationChallenge as Akamai uses Adobe Alchemy
akamai_pv = manifest.find('{http://ns.adobe.com/f4m/1.0}pv-2.0') akamai_pv = manifest.find('{http://ns.adobe.com/f4m/1.0}pv-2.0')
if akamai_pv is not None and ';' in akamai_pv.text: if akamai_pv is not None and ';' in akamai_pv.text:
@ -1036,8 +1035,16 @@ class InfoExtractor(object):
'bootstrap info', default=None) 'bootstrap info', default=None)
for i, media_el in enumerate(media_nodes): for i, media_el in enumerate(media_nodes):
if manifest_version == '2.0' or assume_f4mv2: # If <bootstrapInfo> is present, the specified f4m is a
media_url = media_el.attrib.get('href') or media_el.attrib.get('url') # stream-level manifest, and only set-level manifests may refer to
# external resources. See section 11.4 and section 4 of F4M spec
if bootstrap_info is None:
media_url = None
# @href is introduced in 2.0, see section 11.6 of F4M spec
if manifest_version == '2.0':
media_url = media_el.attrib.get('href')
if media_url is None:
media_url = media_el.attrib.get('url')
if not media_url: if not media_url:
continue continue
manifest_url = ( manifest_url = (

View file

@ -60,7 +60,7 @@ class PlaywireIE(InfoExtractor):
thumbnail = content.get('poster') thumbnail = content.get('poster')
src = content['media']['f4m'] src = content['media']['f4m']
formats = self._extract_f4m_formats(src, video_id, assume_f4mv2=True, m3u8_id='hls') formats = self._extract_f4m_formats(src, video_id, m3u8_id='hls')
for a_format in formats: for a_format in formats:
if not dict_get(a_format, ['tbr', 'width', 'height']): if not dict_get(a_format, ['tbr', 'width', 'height']):
a_format['quality'] = 1 if '-hd.' in a_format['url'] else 0 a_format['quality'] = 1 if '-hd.' in a_format['url'] else 0