MDL-64452 cache: Always use the current session id in the key prefix
authorDavid Mudrák <david@moodle.com>
Tue, 8 Jan 2019 18:23:59 +0000 (19:23 +0100)
committerDavid Mudrák <david@moodle.com>
Fri, 15 Feb 2019 08:36:43 +0000 (09:36 +0100)
commit0ea0dfde6de4674e62b3b630d2e41023e2965db2
tree11fd163058fd0ef585e431e2f96a2527743c6029
parent124999563ab5e235d31fdecafcd41eb45e3b21e3
MDL-64452 cache: Always use the current session id in the key prefix

The current logic in the cache_session::check_tracked_user() is not
right. We must always set the current session id. A typical use case is
when the cache instance is instantiated for a not logged in user. We
can't let the sessionid property null in that case as it forms an
important part of the parsed key.

Similarly, even if we have the same user currently loaded, we must still
set the sessionid to make sure the data will be associated with the
current PHP session. Same user (including visitors or guest users) can
access the site from different browsers and each must end up with its
own key prefix.
cache/classes/loaders.php