Allow redirecting when not logged in
Summary: Without this, the https redirect doesn't work if you're not logged in, because the login check in willBeginExecution happens before the redirect controller can redirect. This also has the unpleasant effect of the login page on http (when it should have redirected to https) not having any css or js. Test Plan: With the https redirect enabled, loaded phabricator without being logged in, and saw that I got redirected to the https login page instead of seeing a http login page. Reviewers: epriestley, vrana Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D3438
This commit is contained in:
@@ -20,6 +20,14 @@ final class PhabricatorRedirectController extends PhabricatorController {
|
||||
|
||||
private $uri;
|
||||
|
||||
public function shouldRequireLogin() {
|
||||
return false;
|
||||
}
|
||||
|
||||
public function shouldRequireEnabledUser() {
|
||||
return false;
|
||||
}
|
||||
|
||||
public function willProcessRequest(array $data) {
|
||||
$this->uri = $data['uri'];
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user