diff options
author | Sergey M․ <dstftw@gmail.com> | 2018-01-16 22:34:16 +0700 |
---|---|---|
committer | Sergey M․ <dstftw@gmail.com> | 2018-01-16 22:34:16 +0700 |
commit | 1370dba59f140dc5313ed1da1dad7bcba09cfb1c (patch) | |
tree | 26b4d62c8c2f66e8bf0c358d09804bcf532a8d6b /youtube_dl/extractor/twitch.py | |
parent | 1d1d60f6dd4d7c33dc690ff5c68fdab11e6c0af7 (diff) |
[twitch] Fix authentication and error capture (closes #14090, closes #15264)
Diffstat (limited to 'youtube_dl/extractor/twitch.py')
-rw-r--r-- | youtube_dl/extractor/twitch.py | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/youtube_dl/extractor/twitch.py b/youtube_dl/extractor/twitch.py index f9164af09..1981b4d4a 100644 --- a/youtube_dl/extractor/twitch.py +++ b/youtube_dl/extractor/twitch.py @@ -85,10 +85,15 @@ class TwitchBaseIE(InfoExtractor): if isinstance(e.cause, compat_HTTPError) and e.cause.code == 400: response = self._parse_json( e.cause.read().decode('utf-8'), None) - fail(response['message']) + fail(response.get('message') or response['errors'][0]) raise - redirect_url = urljoin(post_url, response['redirect']) + if 'Authenticated successfully' in response.get('message', ''): + return None, None + + redirect_url = urljoin( + post_url, + response.get('redirect') or response['redirect_path']) return self._download_webpage_handle( redirect_url, None, 'Downloading login redirect page', headers=headers) @@ -106,6 +111,10 @@ class TwitchBaseIE(InfoExtractor): 'password': password, }) + # Successful login + if not redirect_page: + return + if re.search(r'(?i)<form[^>]+id="two-factor-submit"', redirect_page) is not None: # TODO: Add mechanism to request an SMS or phone call tfa_token = self._get_tfa_info('two-factor authentication token') |