-
Notifications
You must be signed in to change notification settings - Fork 7
Check for empty vault password. Closes #444. #457
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, left some questions, feel free to merge if no changes are required.
rho/vault.py
Outdated
@@ -23,6 +23,8 @@ | |||
from rho.translation import _ as t | |||
|
|||
PROMPT = "Please enter your rho vault password: " | |||
ERROR_PROMPT = 'Error: The vault password cannot be empty. '\ | |||
'Please provide a none empty password.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure but I think the message should be "Please provide a non empty password", mind non instead of none?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, I'll fix it. should be non-empty
.
vault_password = '' | ||
while vault_password == '': | ||
vault_password = getpass("\033[01;36m" + prompt + "\033[0m") | ||
if vault_password == '': |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doing this validation it would allow one or more spaces as a vault password, is it ok?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've verified that spaces are fine as a password.
1 similar comment
* master: Add jboss.eap.init-files to JBOSS_FACTS. Closes #478. (#479) Add default dictionaries to handle lost host. Handle ansible rc if csv was successfully created. Closes #475. (#476) Fix addition of certs for wrong when condition. Closes #472. (#474) Present more status during discovery for larger environments. Closes #442. (#467) revert default logging change. Closes #460. (#461) Check for empty vault password. Closes #444. (#457) Highlight potential issues in the console output. Closes #443. (#458) Default verbosity to 1 for improved log data. Closes #441. (#456) Turn off colors in the ansible log. Closes #448. (#455) Fix task that can hang if systemctl paginates Update playbook to use free strategy. Closes #445. (#453) Merge 0.30 version into master branch (#451) # Conflicts: # rho/ansible_utils.py # rho/facts.py # rho/inventory_scan.py # rho/postprocessing.py # rho/scancommand.py # roles/jboss_eap/tasks/main.yml # roles/write/tasks/main.yml # test/test_postprocessing.py
* Merge branch 'master' into dev * master: Add jboss.eap.init-files to JBOSS_FACTS. Closes #478. (#479) Add default dictionaries to handle lost host. Handle ansible rc if csv was successfully created. Closes #475. (#476) Fix addition of certs for wrong when condition. Closes #472. (#474) Present more status during discovery for larger environments. Closes #442. (#467) revert default logging change. Closes #460. (#461) Check for empty vault password. Closes #444. (#457) Highlight potential issues in the console output. Closes #443. (#458) Default verbosity to 1 for improved log data. Closes #441. (#456) Turn off colors in the ansible log. Closes #448. (#455) Fix task that can hang if systemctl paginates Update playbook to use free strategy. Closes #445. (#453) Merge 0.30 version into master branch (#451) # Conflicts: # rho/ansible_utils.py # rho/facts.py # rho/inventory_scan.py # rho/postprocessing.py # rho/scancommand.py # roles/jboss_eap/tasks/main.yml # roles/write/tasks/main.yml # test/test_postprocessing.py
No description provided.