Do not echo command with password - Githubissues

cyberark / cyberark-conjur-cli

CyberArk Conjur command line interface written in Python

https://www.conjur.org

Apache License 2.0

17 stars 16 forks source link

Do not echo command with password #315

Closed orenbm closed 3 years ago

orenbm commented 3 years ago

What does this PR do?

Although the password is generated, we should not print it to the log as we may change it in the future to a real secret (e.g the one from conjurops)

Checklists

Change log

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[x] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
[x] This PR does not require updating any documentation