Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Critical
-
Resolution: Fixed
-
Affects Version/s: 2.7.16, 2.9.8, 3.0.6, 3.1.2
-
Component/s: Authentication
-
Testing Instructions:
-
Affected Branches:MOODLE_27_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE
-
Fixed Branches:MOODLE_27_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE
-
Pull from Repository:
-
Pull Master Branch:
MDL-55945-master -
Pull Master Diff URL:
Description
Reported here. Looks like I missed https support in my patch for MDL-50803.
Suggested patch:
change:
redirect($CFG->wwwroot . '/login/forgot_password.php'); |
to:
if ($CFG->loginhttps) { |
redirect($CFG->httpswwwroot . '/login/forgot_password.php'); |
} else { |
redirect($CFG->wwwroot . '/login/forgot_password.php'); |
}
|