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

daily show download not working #4447

Closed
pkhannah opened this issue Dec 12, 2014 · 2 comments
Closed

daily show download not working #4447

pkhannah opened this issue Dec 12, 2014 · 2 comments

Comments

@pkhannah
Copy link

@pkhannah pkhannah commented Dec 12, 2014

This happens with the default ubuntu install and the latest as of now in github.

Fails: http://thedailyshow.cc.com/full-episodes/tjnjbd/december-10--2014---suki-kim
Works: http://thedailyshow.cc.com/full-episodes/mjgsgz/december-9--2014---kathryn-bigelow---juan-zarate

Message:
[ComedyCentralShows] december-10--2014---suki-kim: Downloading webpage
[ComedyCentralShows] december-10--2014---suki-kim: Downloading show index
[ComedyCentralShows] december-10--2014---suki-kim: Downloading configuration for segment 1 / 4
Traceback (most recent call last):
File "/home/pkh/youtube-dl/bin/youtube-dl", line 6, in
youtube_dl.main()
File "/usr/lib/python2.7/dist-packages/youtube_dl/init.py", line 355, in main
_real_main(argv)
File "/usr/lib/python2.7/dist-packages/youtube_dl/init.py", line 345, in _real_main
retcode = ydl.download(all_urls)
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1121, in download
res = self.extract_info(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 553, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 241, in extract
return self._real_extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/comedycentral.py", line 188, in _real_extract
'Downloading configuration for segment %d / %d' % (part_num + 1, len(item_els)))
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 382, in _download_xml
return xml.etree.ElementTree.fromstring(xml_string.encode('utf-8'))
File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1300, in XML
parser.feed(text)
File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
self._raiseerror(v)
File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
raise err
xml.etree.ElementTree.ParseError: syntax error: line 1, column 49

@pkhannah
Copy link
Author

@pkhannah pkhannah commented Dec 12, 2014

K, not an issue with the app as such.

data: '\n\n<title>503 Service Temporarily Unavailable</title>\n\n

Service Temporarily Unavailable

\n

The server is temporarily unable to service your\nrequest due to maintenance downtime or capacity\nproblems. Please try again later.

\n'

But, maybe it's possible to catch this so we can display this text (possibly sanitized?) to the user so it's obvious where the problem is?

@phihag
Copy link
Contributor

@phihag phihag commented Dec 12, 2014

Works now. Since this is a "soft 503" (i.e. the HTTP code is 200) and only happened once, I do not think it's necessary to add dedicated handler code for this rare condition, although you are certainly welcome to contribute.

Thanks for the report!

@phihag phihag closed this Dec 12, 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.