LiveUser - session_set_cookie_params()
| From: | Koistya `Navin | Date: | Thu, 07 Jul 2005 01:43:04 +0000 |
| Subject: | LiveUser - session_set_cookie_params() | ||
| Groups: | php.pear.dev | ||
| Request: | Send a blank email to pear-dev+get-38469@lists.php.net to get a copy of this message | ||
I thinks there is a bug in session_set_cookie_params() - LiveUser.php, line:
906
At least this way it doesn't work with IE.
Original code:
if ($this->_options['session_cookie_params']) {
session_set_cookie_params((
time() + (LIVEUSER_DAY_SECONDS *
$this->_options['session_cookie_params']['lifetime'])),
$this->_options['session_cookie_params']['path'],
$this->_options['session_cookie_params']['domain'],
$this->_options['session_cookie_params']['secure']);
}
Change suggestion:
if ($this->_options['session_cookie_params']) {
session_set_cookie_params(
(60 * 60 * 24 *
$this->_options['session_cookie_params']['lifetime']),
$this->_options['session_cookie_params']['path'],
$this->_options['session_cookie_params']['domain'],
$this->_options['session_cookie_params']['secure']);
}