Summary | sql based permissions cache expire bug with save |
Queue | Horde Base |
Queue Version | 3.3.10 |
Type | Bug |
State | Resolved |
Priority | 2. Medium |
Owners | jan (at) horde (dot) org |
Requester | vamos (at) netikka (dot) fi |
Created | 11/16/2010 (5343 days ago) |
Due | |
Updated | 11/25/2010 (5334 days ago) |
Assigned | |
Resolved | 11/25/2010 (5334 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | 3.3.12 |
Patch | Yes |
State ⇒ Resolved
Milestone ⇒ 3.3.12
Fix inconsistent cache keys (
Bug #9392).http://cvs.horde.org/diff.php/framework/Perms/Perms/Attic/sql.php?rt=horde&r1=1.1.2.16&r2=1.1.2.17&ty=u
http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horde&r1=1.515.2.642&r2=1.515.2.643&ty=u
Priority ⇒ 2. Medium
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Horde Base
Summary ⇒ sql based permissions cache expire bug with save
Type ⇒ Bug
State ⇒ Unconfirmed
class save method is incorrectly expiring wrong cache entry
$cache->expire('perm_sql_' . $name):
# diff -u horde-3.3.10/lib/Horde/Perms/sql.php
horde-3.3.10-new/lib/Horde/Perms/sql.php
--- horde-3.3.10/lib/Horde/Perms/sql.php 2010-10-25
15:58:55.000000000 +0300
+++ horde-3.3.10-new/lib/Horde/Perms/sql.php 2010-11-16
12:05:41.000000000 +0200
@@ -524,7 +524,7 @@
}
$cache =
Horde_Cache::singleton($GLOBALS['conf']['cache']['driver'],
Horde::getDriverConfig('cache', $GLOBALS['conf']['cache']['driver']));
- $cache->expire('perm_sql_' . $name);
+ $cache->expire('perm_sql' . $name);
$cache->expire('perm_sql_exists_' . $name);
return true;