We found a similar problem in the function setcookie() in "program/include/rcmail.php":
public static function setcookie($name, $value, $exp = 0) { $cookie = session_get_cookie_params(); setcookie($name, $value, $exp, $cookie['path'], $cookie['domain'], ($_SERVER['HTTPS'] && ($_SERVER['HTTPS'] != 'off'))); }
A boolean variable, for example $secure, should be determined before calling setcookie(). The boolean variable could be set according to the same logic as in "iniset.php" in trunc 2211.
Regards, Jonas
On Fri, Jan 2, 2009 at 3:27 PM, A.L.E.C alec@alec.pl wrote:
The existence of $_SERVER['HTTPS'] should be tested before any attempt to retrieve its value.
Thanks, but we just need to move upper the line: ini_set('error_reporting', E_ALL&~E_NOTICE);
We need to fix it. Instead of hiding it since most of those errors are being handled more strict in later PHP versions. Besides, a side effect is cleaner and more robust code.
http://trac.roundcube.net/changeset/2211/trunk/roundcubemail/program
List info: http://lists.roundcube.net/dev/