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

lib/weblib.php::get_file_argument(): add support for $_SERVER['ORIG_PATH_INFO'] when $_SERVER['PATH_INFO'] is missing

    XMLWordPrintable

    Details

    • Type: New Feature
    • Status: Reopened
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.0, 2.1
    • Fix Version/s: DEV backlog
    • Component/s: Files API
    • Labels:
    • Environment:
      Apache, missing PATH_INFO but providing ORIG_PATH_INFO
    • Testing Instructions:
      Hide

      Note: it requires the availability of a "broken" environment.

      Show
      Note: it requires the availability of a "broken" environment.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE

      Description

      See http://moodle.org/mod/forum/discuss.php?d=181931 for the long story.

      Shortly: sometimes there are situations in which ORIG_PATH_INFO env variable plays the role of the missing PATH_INFO: among them some shared server offerings from Hosting Companies like 1&1. This kind of setup will prevent Moodle to correctly provide raw assets using slash arguments, especially when it is warmly recommended like for the AICC/SCORM activity.

      The patch proposal in http://moodle.org/mod/forum/discuss.php?d=181931#p792671 gives the opportunity to mask the limitations of such a server environment. This proposal could be evaluated in terms of correctness&regression and of the other parts of the core code in which ORIG_PATH_INFO is taken into account (e.g. external libraries) or where PATH_INFO is straight used.

      For the sake of simplicity, the patch proposal focuses to the main blocking issue, the inability of serving files of a AICC/SCORM package and nothing more.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              matteo Matteo Scaramuccia
              Participants:
              Component watchers:
              Matteo Scaramuccia, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              2 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated: