Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-38565

When a cache store is set to MODE_SESSION it should include the userid in the keys.

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 2.4.3, 2.5
    • 2.4.4, 2.5
    • Caching
    • MOODLE_24_STABLE, MOODLE_25_STABLE
    • MOODLE_24_STABLE, MOODLE_25_STABLE
    • wip-MDL-38565-m25
    • Hide
      1. Run unit tests.
      2. Run them again - there are lots!
      3. Upgrade a site
      4. Perform a fresh install
      5. Log in as an admin and set up the front page to display all options for all users.
      6. Test the front page as a guest, student, and admin.
      7. Test switching users.
      8. Run performance tests if you can.
      Show
      Run unit tests. Run them again - there are lots! Upgrade a site Perform a fresh install Log in as an admin and set up the front page to display all options for all users. Test the front page as a guest, student, and admin. Test switching users. Run performance tests if you can.

    Description

      Otherwise a site configured with a shared mem storage for the session cache will have a shared keyspace for all users which is not expected.

      Comments from Sam:
      It should be in the loader
      (10:30:09) samhemelryk@moodle.org: cache_session::parse_key should be doing it, but the code is not there ...

      ... write unit tests to test the stores explicitly for collision across all modes is perhaps the best way

      Attachments

        Issue Links

          Activity

            People

              samhemelryk Sam Hemelryk
              damyon Damyon Wiese
              Damyon Wiese Damyon Wiese
              Dan Poltawski Dan Poltawski
              Adrian Greeve Adrian Greeve
              Matteo Scaramuccia, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                13/May/13