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:
                11/May/20

                Time Tracking

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