Menu
▾
▴
#2826 configtest failes when executed externally due to not set variables
using configtest.php externally (with $allow_remote_configtest = true) fails with error message:
Notice: Undefined variable: client_ip in /srv/www/de/realmo/webmail/docs/src/configtest.php on line 655
Notice: Undefined variable: client_ip in /srv/www/de/realmo/webmail/docs/src/configtest.php on line 658
FATAL ERROR: SMTP EHLO failed. You need ESMTP support for SMTP STARTTLS
SquirrelMail version: 1.5.2 [SVN] (from 2015-01-23)
proposed fix:
move the lines 178+179 to before the check of $allow_remote_configtest
before:
/ Block remote use of script /
if (! $allow_remote_configtest) {
sqGetGlobalVar('REMOTE_ADDR',$client_ip,SQ_SERVER);
sqGetGlobalVar('SERVER_ADDR',$server_ip,SQ_SERVER);
if ((! isset($client_ip) || ...
<<
fixed:
sqGetGlobalVar('REMOTE_ADDR',$client_ip,SQ_SERVER);
sqGetGlobalVar('SERVER_ADDR',$server_ip,SQ_SERVER);
/ Block remote use of script /
if (! $allow_remote_configtest) {
if ((! isset($client_ip) || ...
<<
Committed fix. Thank you!