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

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

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

          Attachments

            Activity

            Hide
            quen 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
            quen 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
            skodak Petr Skoda added a comment -

            thanks for noticing it

            Show
            skodak 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:
                  Fix Release Date:
                  31/Mar/07