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

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  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
                  Permalink
                  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