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

Strange error when downloading from "youtube.com/watch?v" #14718

Closed
ScttB opened this issue Nov 11, 2017 · 2 comments
Closed

Strange error when downloading from "youtube.com/watch?v" #14718

ScttB opened this issue Nov 11, 2017 · 2 comments

Comments

@ScttB
Copy link

@ScttB ScttB commented Nov 11, 2017

>youtube-dl https://www.youtube.com/watch?v
[generic] watch?v: Requesting header
[redirect] Following redirect to https://www.youtube.com/
[generic] www.youtube: Requesting header
WARNING: Falling back on generic information extractor.
[generic] www.youtube: Downloading webpage
[generic] www.youtube: Extracting information
ERROR: Unsupported URL: https://www.youtube.com/

What I think should happen instead of this is

>youtube-dl https://www.youtube.com/watch?v
ERROR: Video not set - check URL for errors

I believe this behavior should be implemented is because if you use youtube-dl in a batch script, this error might happen because of the = being ignored by a parsing error if the URL is not enclosed in double-quotations when invoked.

Possibly you could check if the program is running from a batch script on Windows and return ERROR: Put URL in double-quotes?

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Nov 11, 2017

Controlling proper escaping or quoting is responsibility of a user not youtube-dl. Needless to say it's technically impossible to know whether the original URL contained unescaped & or not.

@dstftw dstftw closed this Nov 11, 2017
@ScttB
Copy link
Author

@ScttB ScttB commented Nov 11, 2017

I didn't ask to control escaping. I asked for the error to be more informative, assuming it might have been ruined by escaping or user error.
Edit: Also, I was not talking about &, I was talking about =.

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.