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

Redirect to https if it's required (backport of MDL-58318)

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Add a self signed certificate to apache. debian hint: "sudo make-ssl-cert generate-default-snakeoil --force-overwrite".

      Change moodle wwwroot to https version

      Access the site via http - not https

      Verify you are redirected to the https front page and do not see an error.

      Try and access the login/token.php page via http. You should see an error and not be redirected.

      Show
      Add a self signed certificate to apache. debian hint: "sudo make-ssl-cert generate-default-snakeoil --force-overwrite". Change moodle wwwroot to https version Access the site via http - not https Verify you are redirected to the https front page and do not see an error. Try and access the login/token.php page via http. You should see an error and not be redirected.
    • Affected Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Pull from Repository:

      Description

      Without this change, admins are expected to implement complicated redirects in the webserver and sometimes they will not even have access to make these changes.

      We should always have done this IMO.

        Attachments

          Issue Links

          has been marked as being related by

          Improvement - An enhancement to an existing Moodle feature. MDL-58318 https is forced, but no redirect to https URL is provided.

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              • Assignee:
                damyon Damyon Wiese
                Reporter:
                damyon Damyon Wiese
                Integrator:
                Eloy Lafuente (stronk7)
                Tester:
                Eloy Lafuente (stronk7)
                Participants:
                Component watchers:
                Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  13/Nov/17