Closed (fixed)
Project:
Authenticated User Page Caching (Authcache)
Version:
7.x-2.x-dev
Component:
Documentation
Priority:
Minor
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Dec 2015 at 19:54 UTC
Updated:
29 Nov 2017 at 14:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
znerol commentedThis probably is not obvious from the documentation, but the cache key must not vary per path. The only safe thing to do in this situation is to unconditionally add the user-terms to the key properties. As a result a different key will be generated for each combination of taxonomy terms (and roles).
Comment #3
znerol commentedAttached is a patch with docs update. I tried to phrase it in a generic way but I'm not sure whether it is clear enough.
Comment #4
angel.angelio commentedIMO, I'd add @see authcache_builtin_cacheinc_retrieve_cache_page
So if the key does not change that often - I mean a 1 or 2 times per session- it might be "OK" to do it.
Comment #5
znerol commentedFrankly I'm not quite sure.
Comment #6
znerol commentedThis patch should be included into a release regardless of the open question.
Comment #8
znerol commentedCareful with hand-editing patches ;)
Comment #9
znerol commentedComment #11
znerol commented