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

DRTV new/changed timestamp format not detected #4059

Closed
bangert opened this issue Oct 29, 2014 · 3 comments
Closed

DRTV new/changed timestamp format not detected #4059

bangert opened this issue Oct 29, 2014 · 3 comments

Comments

@bangert
Copy link

@bangert bangert commented Oct 29, 2014

bangert@gaston ~/data/video/true-detective $ youtube-dl http://www.dr.dk/tv/se/true-detective/true-detective-2-4
[DRTV] true-detective-2-4: Downloading video JSON
Traceback (most recent call last):
File "/usr/bin/youtube-dl", line 6, in
youtube_dl.main()
File "/usr/lib64/python2.7/site-packages/youtube_dl/init.py", line 352, in main
_real_main(argv)
File "/usr/lib64/python2.7/site-packages/youtube_dl/init.py", line 342, in _real_main
retcode = ydl.download(all_urls)
File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 1078, in download
res = self.extract_info(url)
File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 529, in extract_info
ie_result = ie.extract(url)
File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/common.py", line 197, in extract
return self._real_extract(url)
File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/drtv.py", line 38, in _real_extract
timestamp = parse_iso8601(data['CreatedTime'][:-5])
File "/usr/lib64/python2.7/site-packages/youtube_dl/utils.py", line 942, in parse_iso8601
dt = datetime.datetime.strptime(date_str, date_format) - timezone
File "/usr/lib64/python2.7/_strptime.py", line 325, in _strptime
(data_string, format))
ValueError: time data '2014-10-17T23:3' does not match format '%Y-%m-%dT%H:%M:%S'

this does not affect all streams, as http://www.dr.dk/tv/se/boern/ramasjang/skaeg-med-matematik/skaeg-med-matematik-6 works fine.

Thanks!

@bangert
Copy link
Author

@bangert bangert commented Oct 29, 2014

this is youtube-dl version 2014.10.27-gentoo_no_offensive_sites

@phihag phihag closed this in 6ad4013 Oct 29, 2014
@phihag
Copy link
Contributor

@phihag phihag commented Oct 30, 2014

Thank you for the report. This has been fixed in youtube-dl 2014.10.30. See our FAQ on how to update.

@bangert
Copy link
Author

@bangert bangert commented Nov 3, 2014

awesome - thank you very much.

Den 30-10-2014 15:51, Philipp Hagemeister skrev:

Thank you for the report. This has been fixed in youtube-dl
2014.10.30. See our FAQ
https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-update-youtube-dl
on how to update.


Reply to this email directly or view it on GitHub
#4059 (comment).

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.