From 9b1d2e93b81d404815361398cea7a62bf84041e9 Mon Sep 17 00:00:00 2001 From: Jakub Zelenka Date: Sat, 30 Mar 2024 14:10:31 +0000 Subject: [PATCH] Fix FPM tester default env vars change This broke cloexec test --- sapi/fpm/tests/tester.inc | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/sapi/fpm/tests/tester.inc b/sapi/fpm/tests/tester.inc index ff834c329fa..ed0988f883a 100644 --- a/sapi/fpm/tests/tester.inc +++ b/sapi/fpm/tests/tester.inc @@ -417,11 +417,12 @@ class Tester /** * Start PHP-FPM master process * - * @param array $extraArgs Command extra arguments. - * @param bool $forceStderr Whether to output to stderr so error log is used. - * @param bool $daemonize Whether to start FPM daemonized - * @param array $extensions List of extension to add if shared build used. - * @param array $iniEntries List of ini entries to use. + * @param array $extraArgs Command extra arguments. + * @param bool $forceStderr Whether to output to stderr so error log is used. + * @param bool $daemonize Whether to start FPM daemonized + * @param array $extensions List of extension to add if shared build used. + * @param array $iniEntries List of ini entries to use. + * @param array|null $envVars List of env variable to execute FPM with or null to use the current ones. * * @return bool * @throws \Exception @@ -432,7 +433,7 @@ class Tester bool $daemonize = false, array $extensions = [], array $iniEntries = [], - array $envVars = [], + ?array $envVars = null, ) { $configFile = $this->createConfig(); $desc = $this->outDesc ? [] : [1 => array('pipe', 'w'), 2 => array('redirect', 1)];