Moodle
  1. Moodle
  2. MDL-7699

Relative redirect() does not work with ../../

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 1.8
        • Fix Version/s: 1.8
        • Component/s: Libraries
        • Labels:
          None
        • Affected Branches:
          MOODLE_18_STABLE
        • Fixed Branches:
          MOODLE_18_STABLE

          Description

          The redirect() function includes a clever regular expression that I and Petr wrote. It isn't clever enough to work right when you do ../../ (i.e. more than one ../) though

            Gliffy Diagrams

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Sam Marshall added a comment -

              OK, I have checked in a fix into HEAD. I don't believe there is any code in Moodle that relies on this ability (redirect with more than one level of ..) yet so it's probably unnecessary to merge this into stable.

              Show
              Sam Marshall added a comment - OK, I have checked in a fix into HEAD. I don't believe there is any code in Moodle that relies on this ability (redirect with more than one level of ..) yet so it's probably unnecessary to merge this into stable.
              Hide
              Permalink
              Petr Skoda added a comment -

              thanks for noticing it

              Show
              Petr Skoda added a comment - thanks for noticing it

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: