Submitted by crystalcross on Thu, 01/21/2016 - 19:09
Webmin ver 1.782
There is an issue creating new servers with atomic-php installed. The reference to the php in virtual-server/php-lib.pl is incorrect.
By default it references "php" but it should reference "php-cgi" instead. Servers were building correctly, but php was crashing with an fcgi timeout error.
Changing the reference from php to php-cgi corrected the problem.
Original Code:
if ($nodotv ne $v) {
# For a version like 5.4, check for binaries like php54 and
# /opt/rh/php54/root/usr/bin/php
$phpn ||= &has_command("php$nodotv-cgi") ||
&has_command("php-cgi$nodotv") ||
&has_command("/opt/rh/php$nodotv/root/usr/bin/php-cgi") ||
&has_command("/opt/rh/rh-php$nodotv/root/usr/bin/php-cgi") ||
&has_command("/opt/atomic/atomic-php$nodotv/root/usr/bin/php") ||
&has_command("/opt/rh/php$nodotv/bin/php-cgi") ||
&has_command("/opt/remi/php$nodotv/root/usr/bin/php-cgi") ||
&has_command("php$nodotv") ||
&has_command("/opt/rh/php$nodotv/root/usr/bin/php");
&has_command("/opt/rh/rh-php$nodotv/root/usr/bin/php");
&has_command("/opt/rh/php$nodotv/bin/php") ||
&has_command(glob("/opt/phpfarm/inst/bin/php-cgi-$v.*"));
Updated Code:
if ($nodotv ne $v) {
# For a version like 5.4, check for binaries like php54 and
# /opt/rh/php54/root/usr/bin/php
$phpn ||= &has_command("php$nodotv-cgi") ||
&has_command("php-cgi$nodotv") ||
&has_command("/opt/rh/php$nodotv/root/usr/bin/php-cgi") ||
&has_command("/opt/rh/rh-php$nodotv/root/usr/bin/php-cgi") ||
&has_command("/opt/atomic/atomic-php$nodotv/root/usr/bin/php-cgi") ||
&has_command("/opt/rh/php$nodotv/bin/php-cgi") ||
&has_command("/opt/remi/php$nodotv/root/usr/bin/php-cgi") ||
&has_command("php$nodotv") ||
&has_command("/opt/rh/php$nodotv/root/usr/bin/php");
&has_command("/opt/rh/rh-php$nodotv/root/usr/bin/php");
&has_command("/opt/rh/php$nodotv/bin/php") ||
&has_command(glob("/opt/phpfarm/inst/bin/php-cgi-$v.*"));Status:
Closed (fixed)
Comments
Submitted by JamieCameron on Thu, 01/21/2016 - 21:11 Comment #1
Thanks, I will include that fix in the next release.
Submitted by JamieCameron on Thu, 01/21/2016 - 21:11 Comment #2