Fix the replacement of the credentials.

://username:password@domain
was replaced by ://://:***@domain
instead of ://username:***@domain or ://***:***@domain
main
Stefan Grootscholten 8 years ago
parent ea11a95359
commit 42f4d344a4

@ -45,11 +45,11 @@ class ProcessExecutor
{ {
if ($this->io && $this->io->isDebug()) { if ($this->io && $this->io->isDebug()) {
$safeCommand = preg_replace_callback('{(://)(?P<user>[^:/\s]+):(?P<password>[^@\s/]+)}i', function ($m) { $safeCommand = preg_replace_callback('{(://)(?P<user>[^:/\s]+):(?P<password>[^@\s/]+)}i', function ($m) {
if (preg_match('{^[a-f0-9]{12,}$}', $m[1])) { if (preg_match('{^[a-f0-9]{12,}$}', $m[2])) {
return '://***:***'; return '://***:***';
} }
return '://'.$m[1].':***'; return '://'.$m[2].':***';
}, $command); }, $command);
$this->io->writeError('Executing command ('.($cwd ?: 'CWD').'): '.$safeCommand); $this->io->writeError('Executing command ('.($cwd ?: 'CWD').'): '.$safeCommand);
} }

Loading…
Cancel
Save