Tickets :: Comment on [#1302] incorrect cookie expiration times

* Your Email Address
* Spam protection	Enter the letters below: . .. .. .. .. \|\ \|\|__\|\|\/\|\| \|\| \| \\|\| \|\| \|\|/\\|\|___
Comment	> In the file lib/Horde/Secret.php you use many times: > > > > @setcookie($keyname . '_key', $key, $conf['session']['timeout'], > $conf['cookie']['path'], $conf['cookie']['domain'], $conf > > ['use_ssl'] == 1 ? 1 : 0); > > > > But the setcookie function expect absolute time of cookie expiration, > not relative. Therefore the correct form should be: > > > > @setcookie($keyname . '_key', $conf['session']['timeout'] == 0 ? 0 : > $key, time() + $conf['session']['timeout'], $conf['cookie']['path'], > $conf['cookie']['domain'], $conf > > ['use_ssl'] == 1 ? 1 : 0); > > > > Am I right? I lost about 5 hours :-/ to find a problem why the IMP > application forever reloads when I use cookies. Only MSIE worked (it > kept the cookie with the time in the past).
Attachment
Watch this ticket