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

Support the X-Redirect-By header

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      1) Curl any page which sends a redirect eg:

      curl -v http://moodle.local/admin/

       

      Look for the header returned:

      < HTTP/1.1 303 See Other 
      < Date: Thu, 28 Nov 2019 11:47:37 GMT 
      < Server: Apache/2.4.29 (Ubuntu) 
      ...
      < X-Redirect-By: Moodle 
      < Location: http://moodle.local/admin/index.php?cache=1
      

      Show
      1) Curl any page which sends a redirect eg: curl -v http://moodle.local/admin/   Look for the header returned: < HTTP/1.1 303 See Other < Date: Thu, 28 Nov 2019 11:47:37 GMT < Server: Apache/2.4.29 (Ubuntu) ... < X-Redirect-By: Moodle < Location: http://moodle.local/admin/index.php?cache=1
    • Affected Branches:
      MOODLE_39_STABLE
    • Fixed Branches:
      MOODLE_39_STABLE
    • Pull Master Branch:
      MDL-67403-x-redirect-by

      Description

      This is a super simple defacto spec which was suggested by Yoast and implemented in wordpress.

      https://developer.yoast.com/x-redirect-by-header/

      It helps when you have a complicated stack and are debugging redirect issues and it's not clear from the outside which layer in the stack is performing the redirect. Redirects themselves are pretty light so it's a minor data burden overall. It makes more sense if it's always on and not behind a debugging type switch.

        Attachments

          Activity

            People

            Assignee:
            brendanheywood Brendan Heywood
            Reporter:
            brendanheywood Brendan Heywood
            Peer reviewer:
            Dan Marsden
            Integrator:
            Eloy Lafuente (stronk7)
            Tester:
            Eloy Lafuente (stronk7)
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              15/Jun/20

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 35 minutes
                35m