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

ycrc should prompt for password #2

Open
tylarb opened this issue Oct 26, 2020 · 3 comments
Open

ycrc should prompt for password #2

tylarb opened this issue Oct 26, 2020 · 3 comments
Labels
ycrc YCql Row Count

Comments

@tylarb
Copy link
Contributor

tylarb commented Oct 26, 2020

Currently accepts password, but as command line flag.

@tylarb tylarb added the ycrc YCql Row Count label Oct 26, 2020
@tylarb
Copy link
Contributor Author

tylarb commented Feb 12, 2021

In addition to password accept, a --password-on-stdin would be great

@tylarb
Copy link
Contributor Author

tylarb commented Feb 12, 2021

In fact... --password should become --password-on-stdin and otherwise, if password required, prompt

@tylarb
Copy link
Contributor Author

tylarb commented Feb 12, 2021

Specification...

Assume DB is password protected and this command is executed:

 ./ycrc test -s 1 --tables=mytable2,mytable3  

The query should be attempted, and if there's a connection failure to DB, specifically an AUTH failures, not a pport or network, or general failures, then the ycrc should prompt for a password on the cmdline using hidden type text

this commit should also remove --password and replace with --password-on-stdin which will do the same thing as password but be hidden

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

No branches or pull requests

1 participant