Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

swrmediathek fails on some videos, w/ simple fix. #4130

Closed
andikmu opened this issue Nov 7, 2014 · 1 comment
Closed

swrmediathek fails on some videos, w/ simple fix. #4130

andikmu opened this issue Nov 7, 2014 · 1 comment

Comments

@andikmu
Copy link

@andikmu andikmu commented Nov 7, 2014

Videos with higher resolution fail on swrmediathek.
e.g. :
youtube-dl http://swrmediathek.de/player.htm?show=1f47d590-5f3e-11e4-8d38-0026b975f2e6

[SWRMediathek] 1f47d590-5f3e-11e4-8d38-0026b975f2e6: Downloading video JSON
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "youtube-dl/youtube-dl/main.py", line 18, in
File "youtube-dl/youtube-dl/youtube_dl/init.py", line 351, in main
File "youtube-dl/youtube-dl/youtube_dl/init.py", line 341, in _real_main
File "youtube-dl/youtube-dl/youtube_dl/YoutubeDL.py", line 1085, in download
File "youtube-dl/youtube-dl/youtube_dl/YoutubeDL.py", line 536, in extract_info
File "youtube-dl/youtube-dl/youtube_dl/extractor/common.py", line 199, in extract
File "youtube-dl/youtube-dl/youtube_dl/extractor/swrmediathek.py", line 83, in _real_extract
IndexError: list index out of range

Suggest this as a solution:

--- youtube_dl.old/extractor/swrmediathek.py    2014-09-29 21:58:46.000000000 +0200
+++ youtube_dl/extractor/swrmediathek.py        2014-11-07 22:28:17.952014531 +0100
@@ -80,7 +80,7 @@

             if media_type == 'Video':
                 fmt.update({
-                    'format_note': ['144p', '288p', '544p'][quality-1],
+                    'format_note': ['144p', '288p', '544p', '720p'][quality-1],
                     'vcodec': codec,
                 })
             elif media_type == 'Audio':
@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Nov 9, 2014

Thanks for the report and for fixing it. (I hadn't read this, that's why I asked for an url).

@jaimeMF jaimeMF closed this Nov 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.