From 814921737186e2e9af0798643de39d2c1bdd0cf7 Mon Sep 17 00:00:00 2001 From: Laurens Post Date: Tue, 3 Mar 2015 16:17:50 +0100 Subject: [PATCH] Add --show-password option to ldap:show-config --- apps/user_ldap/command/showconfig.php | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/apps/user_ldap/command/showconfig.php b/apps/user_ldap/command/showconfig.php index ddbc45243f..fbcf8d57de 100644 --- a/apps/user_ldap/command/showconfig.php +++ b/apps/user_ldap/command/showconfig.php @@ -27,6 +27,12 @@ class ShowConfig extends Command { InputArgument::OPTIONAL, 'will show the configuration of the specified id' ) + ->addOption( + 'show-password', + null, + InputOption::VALUE_NONE, + 'show ldap bind password' + ) ; } @@ -44,15 +50,16 @@ class ShowConfig extends Command { $configIDs = $availableConfigs; } - $this->renderConfigs($configIDs, $output); + $this->renderConfigs($configIDs, $output, $input->getOption('show-password')); } /** * prints the LDAP configuration(s) * @param string[] configID(s) * @param OutputInterface $output + * @param bool $withPassword Set to TRUE to show plaintext passwords in output */ - protected function renderConfigs($configIDs, $output) { + protected function renderConfigs($configIDs, $output, $withPassword) { foreach($configIDs as $id) { $configHolder = new Configuration($id); $configuration = $configHolder->getConfiguration(); @@ -62,7 +69,7 @@ class ShowConfig extends Command { $table->setHeaders(array('Configuration', $id)); $rows = array(); foreach($configuration as $key => $value) { - if($key === 'ldapAgentPassword') { + if($key === 'ldapAgentPassword' && !$withPassword) { $value = '***'; } if(is_array($value)) {