From 42f4d344a491cfb48e8c8c7fef4b794bdf9641fc Mon Sep 17 00:00:00 2001 From: Stefan Grootscholten Date: Wed, 8 Jun 2016 21:59:00 +0200 Subject: [PATCH] Fix the replacement of the credentials. ://username:password@domain was replaced by ://://:***@domain instead of ://username:***@domain or ://***:***@domain --- src/Composer/Util/ProcessExecutor.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Composer/Util/ProcessExecutor.php b/src/Composer/Util/ProcessExecutor.php index d2ced064f..709dcedd5 100644 --- a/src/Composer/Util/ProcessExecutor.php +++ b/src/Composer/Util/ProcessExecutor.php @@ -45,11 +45,11 @@ class ProcessExecutor { if ($this->io && $this->io->isDebug()) { $safeCommand = preg_replace_callback('{(://)(?P[^:/\s]+):(?P[^@\s/]+)}i', function ($m) { - if (preg_match('{^[a-f0-9]{12,}$}', $m[1])) { + if (preg_match('{^[a-f0-9]{12,}$}', $m[2])) { return '://***:***'; } - return '://'.$m[1].':***'; + return '://'.$m[2].':***'; }, $command); $this->io->writeError('Executing command ('.($cwd ?: 'CWD').'): '.$safeCommand); }