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

add prompt password option #134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add prompt password option #134

wants to merge 1 commit into from

Conversation

kamiraux
Copy link

@kamiraux kamiraux commented Aug 7, 2014

I added an option to allow prompting for the password instead of having it stored in the config file or put it on the command line. It is way better for security concerns.

It still supports previous behavior without any change.

To use this option:
transmission-remote-cli -c user@host --prompt-password
or
transmission-remote-cli -c user@host --prompt-password --create-config

@fagga
Copy link
Owner

fagga commented Aug 9, 2014

It would be much nicer if it could detect if a password is needed and ask if it's not stored in the config file. What do you think?

@kamiraux
Copy link
Author

Detect if a password is needed? You mean if the password is empty? Or try to connect and ask for the password if ever it fails?
I can probably do this quite easily.

@fagga
Copy link
Owner

fagga commented Aug 12, 2014

I mean this: try to connect and ask for the password if ever it fails

Automation is always a good thing. If you can dynamically request the password instead of failing and having the user dig for the correct option to ask for it, I think the best path is obvious.

@jakem72360
Copy link

@fagga I managed to get it to prompt for a password automatically if one isn't provided, but required. Furthermore, the user can also be prompted for a username, if one wasn't provided in the connect string. I've started a separate pull request for these changes (#177).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants