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
    • Rank:
      27708

      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

        Activity

        Hide
        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
        Petr Škoda added a comment -

        thanks for noticing it

        Show
        Petr Škoda added a comment - thanks for noticing it

          People

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

            Dates

            • Created:
              Updated:
              Resolved: