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

Pluginfile only reads the session to e.g. check auth, no need to lock

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Not a bug
    • 3.9.4
    • None
    • Files API

    Description

      Locking the session in pluginfile.php means that if you visit a page which locks the session and actively does something for maybe even a few seconds any images loaded through pluginfile cause a php process to be waiting on the lock to become available again. This leads to both a bad user experience and worker starvation as a lot of php processes may end up waiting for the session lock if you have many users and many images. Especially since the increased delay in checking for a session lock after 5 seconds means these processes do not even immediately continue when the lock becomes available.

      As a consequence pluginfile.php should not lock the session if this option is available.

      Identified the problem on 3.9.4 but still present in master.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              naderman Nils Adermann
              Matteo Scaramuccia, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: