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

--password does not allow for asking interactively #10075

Closed
jscott0 opened this issue Jul 12, 2016 · 2 comments
Closed

--password does not allow for asking interactively #10075

jscott0 opened this issue Jul 12, 2016 · 2 comments

Comments

@jscott0
Copy link

@jscott0 jscott0 commented Jul 12, 2016

youtube-dl's --help page says

-p, --password PASSWORD Account password. If this option is left out, youtube-dl will ask interactively.

This means that, when the argument to -p (and --password, respectively) is omitted, it is to prompt for the user's password for downloading a video. This is a useful feature to have, as typing a password as plain text into one's shell is insecure, especially if it saves a log of commands run (i.e. GNU Bash).

However, with the current latest stable version of youtube-dl, the following error can be seen when attempting to make youtube-dl prompt for the password.

youtube-dl: error: --password option requires an argument

However, the --help page states that --password should not require an argument so that it can allow one to be prompted for a password. This shows a conflict between youtube-dl's behavior and its documentation.

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Jul 13, 2016

Left out means -p should not be passed in command line at all only -u.

@dstftw dstftw closed this Jul 13, 2016
@phihag
Copy link
Contributor

@phihag phihag commented Jul 13, 2016

Note that the --help says option, not argument. The error message should make that distinction clear, but nevertheless, if you can suggest a better wording, we may use that.

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.