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

chamilo:status shouldn't show password without explicit param #20

Open
ywarnier opened this issue Feb 17, 2014 · 0 comments
Open

chamilo:status shouldn't show password without explicit param #20

ywarnier opened this issue Feb 17, 2014 · 0 comments

Comments

@ywarnier
Copy link
Member

In chamilo:status, we show the database password on the command line straight away.
However, in the db:show_conn_info command, we get:

# chash db:show_conn_info
Are you sure you want to show the database connection info here? (y/N)

...and we need to say "y" to show the info.

What should happen: for the chamilo:status command, there should be a replacement of the password by "****** (use --showpass to show password)" (and obviously that param should exist).

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

Successfully merging a pull request may close this issue.

1 participant