6.0.0-alpha14
7/3/25

[#8626] lib/Horde/SessionHandler/memcache.php
Summary lib/Horde/SessionHandler/memcache.php
Queue Horde Base
Queue Version 3.3.5
Type Bug
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester whatdoyouwant (at) gmail (dot) com
Created 10/06/2009 (5749 days ago)
Due
Updated 10/07/2009 (5748 days ago)
Assigned
Resolved 10/06/2009 (5749 days ago)
Github Issue Link
Github Pull Request
Milestone 3.3.6
Patch No

History
10/07/2009 07:45:15 AM Jan Schneider Milestone ⇒ 3.3.6
 
10/06/2009 10:22:59 PM Michael Slusarz Comment #5
Assigned to Michael Slusarz
State ⇒ Resolved
Reply to this comment
Fixed in 3.3.6.
10/06/2009 10:22:22 PM CVS Commit Comment #4 Reply to this comment
10/06/2009 10:19:40 PM CVS Commit Comment #3 Reply to this comment
10/06/2009 06:40:20 PM whatdoyouwant (at) gmail (dot) com Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ lib/Horde/SessionHandler/memcache.php
Queue ⇒ Horde Base
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
destroy($id) does not return true if successfully deleted the session 
data from memcache.  This can be seen when turning on horde logging to 
DEBUG.  This was also present in 3.3.4



it's checking against false

should check against true like this:

line 253-256 of memcache.php (only line 253 changed to true instead of false):

         if ($result !== true) {

             Horde::logMessage('Failed to delete session (id = ' . $id 
. ')', __FILE__, __LINE__, PEAR_LOG_DEBUG);

             return false;

         }

Saved Queries