From: Sergey M․ Date: Thu, 6 Mar 2014 23:56:48 +0000 (+0700) Subject: [lynda] Fix successful login regex and fix formats extraction (Closes #2520) X-Git-Url: http://git.bitcoin.ninja/?a=commitdiff_plain;h=16ff7ebc7759e7bc4d746a8bf06afa6afd11eccf;p=youtube-dl [lynda] Fix successful login regex and fix formats extraction (Closes #2520) --- diff --git a/youtube_dl/extractor/lynda.py b/youtube_dl/extractor/lynda.py index 6deed27d7..2b086072a 100644 --- a/youtube_dl/extractor/lynda.py +++ b/youtube_dl/extractor/lynda.py @@ -8,7 +8,8 @@ from .common import InfoExtractor from ..utils import ( compat_urllib_parse, compat_urllib_request, - ExtractorError + ExtractorError, + int_or_none, ) @@ -19,7 +20,7 @@ class LyndaIE(SubtitlesInfoExtractor): _LOGIN_URL = 'https://www.lynda.com/login/login.aspx' _NETRC_MACHINE = 'lynda' - _SUCCESSFUL_LOGIN_REGEX = r'My account' + _SUCCESSFUL_LOGIN_REGEX = r'isLoggedIn: true' _TIMECODE_REGEX = r'\[(?P\d+:\d+:\d+[\.,]\d+)\]' ACCOUNT_CREDENTIALS_HINT = 'Use --username and --password options to provide lynda.com account credentials.' @@ -55,13 +56,29 @@ class LyndaIE(SubtitlesInfoExtractor): duration = video_json['DurationInSeconds'] title = video_json['Title'] - formats = [{'url': fmt['Url'], + formats = [] + + fmts = video_json.get('Formats') + if fmts: + formats.extend([ + { + 'url': fmt['Url'], 'ext': fmt['Extension'], 'width': fmt['Width'], 'height': fmt['Height'], 'filesize': fmt['FileSize'], 'format_id': str(fmt['Resolution']) - } for fmt in video_json['Formats']] + } for fmt in fmts]) + + prioritized_streams = video_json.get('PrioritizedStreams') + if prioritized_streams: + formats.extend([ + { + 'url': video_url, + 'width': int_or_none(format_id), + 'format_id': format_id, + } for format_id, video_url in prioritized_streams['0'].items() + ]) self._sort_formats(formats) @@ -179,6 +196,9 @@ class LyndaCourseIE(InfoExtractor): videos = [] (username, _) = self._get_login_info() + # Might want to extract videos right here from video['Formats'] as it seems 'Formats' is not provided + # by single video API anymore + for chapter in course_json['Chapters']: for video in chapter['Videos']: if username is None and video['HasAccess'] is False: