about summary refs log tree commit diff
diff options
context:
space:
mode:
authorSergey M․ <dstftw@gmail.com>2018-01-16 22:34:16 +0700
committerSergey M․ <dstftw@gmail.com>2018-01-16 22:34:16 +0700
commit1370dba59f140dc5313ed1da1dad7bcba09cfb1c (patch)
tree26b4d62c8c2f66e8bf0c358d09804bcf532a8d6b
parent1d1d60f6dd4d7c33dc690ff5c68fdab11e6c0af7 (diff)
downloadyoutube-dl-1370dba59f140dc5313ed1da1dad7bcba09cfb1c.tar.gz
youtube-dl-1370dba59f140dc5313ed1da1dad7bcba09cfb1c.tar.xz
youtube-dl-1370dba59f140dc5313ed1da1dad7bcba09cfb1c.zip
[twitch] Fix authentication and error capture (closes #14090, closes #15264)
-rw-r--r--youtube_dl/extractor/twitch.py13
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')