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

youtube-dl version 2013.11.17 fails #1786

Closed
khendr01 opened this issue Nov 18, 2013 · 4 comments
Closed

youtube-dl version 2013.11.17 fails #1786

khendr01 opened this issue Nov 18, 2013 · 4 comments

Comments

@khendr01
Copy link

@khendr01 khendr01 commented Nov 18, 2013

Script fails the same on both Linux and Mac, both with and without http proxies (on both platforms). (So the http proxy isn't the issue.)

Command line evidence:

113 Penguin$ youtube-dl --verbose http://www.YouTube.com/watch?v=_VyfE4lVqOY
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'http://www.YouTube.com/watch?v=_VyfE4lVqOY']
[debug] youtube-dl version 2013.11.17
[debug] Python version 2.7.3 - Linux-3.5.0-43-lowlatency-i686-with-Ubuntu-12.10-quantal
[debug] Proxy map: {'ftp': 'http://localhost:8118/', 'http': 'http://localhost:8118/', 'https': 'http://localhost:8118/'}
WARNING: Falling back on generic information extractor.
[generic] watch?v=_VyfE4lVqOY: Downloading webpage
[generic] watch?v=_VyfE4lVqOY: Extracting information
ERROR: Unsupported URL: http://www.YouTube.com/watch?v=_VyfE4lVqOY; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type youtube-dl -U to update.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 404, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 129, in extract
return self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 214, in _real_extract
raise ExtractorError(u'Unsupported URL: %s' % url)
ExtractorError: Unsupported URL: http://www.YouTube.com/watch?v=_VyfE4lVqOY; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type youtube-dl -U to update.

114 Penguin$ youtube-dl -U
youtube-dl is up-to-date (2013.11.17)

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Nov 18, 2013

Running youtube-dl 'http://www.youtube.com/watch?v=_VyfE4lVqOY' works (youtube instead of YouTube).

@phihag
Copy link
Contributor

@phihag phihag commented Nov 18, 2013

This has been fixed in youtube-dl v2013.11.18, at least for YouTube. Type sudo youtube-dl -U to update.

@phihag phihag closed this Nov 18, 2013
@khendr01
Copy link
Author

@khendr01 khendr01 commented Nov 18, 2013

Hmm. Thanks for the work-around.

The Domain Name System (DNS) is designed to be case insensitive. So any case for the machine name should work.

Can the youtube-dl team consider putting in a tolower() call for just the machine name part of the URL, so examples like I gave will work again?

Thanks,
Ken Hendrickson

From: Jaime Marquínez Ferrándiz [mailto:notifications@github.com]
Sent: Monday, November 18, 2013 1:52 AM
To: rg3/youtube-dl
Cc: Hendrickson, Kenneth
Subject: Re: [youtube-dl] youtube-dl version 2013.11.17 fails (#1786)

Running youtube-dl 'http://www.youtube.com/watch?v=_VyfE4lVqOY' works (youtube instead of YouTube).


Reply to this email directly or view it on GitHubhttps://github.com//issues/1786#issuecomment-28678433.

phihag added a commit that referenced this issue Nov 18, 2013
@phihag
Copy link
Contributor

@phihag phihag commented Nov 18, 2013

We are well aware that DNS is case-insensitive. The problem is entirely unrelated to DNS. One can call youtube-dl
with a wide variety of arguments. For example, all of the following work:

https://www.youtube.com/watch?v=_VyfE4lVqOY
//youtube.com/watch?v=_VyfE4lVqOY
YouTube.com/watch?v=_VyfE4lVqOY
_VyfE4lVqOY
PLVyfE4lVqOY
ytsearch:Foo/Bar
ytfavs:
:tds

This matching can often be case-insensitive, but sometimes not (for example, we can detect whether something is a youtube video or a youtube playlist ID by looking at the first two characters). Therefore, I am not sure how we would be able to proceed. Feel free to propose a patch/pull though.

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
3 participants
You can’t perform that action at this time.