Skip to content

Commit cb84cb1

Browse files
author
epriestley
committedMar 22, 2013
Don't require admin privileges to verify email addresses
Summary: Fixes T2821. In D5386 we locked most of this app down, but missed one controller which needs to be accessible by non-admins (this controller should probably be in some other app, like auth, in the long term). Test Plan: @shanemhansen confirmed this fixed his install Reviewers: chad, AnhNhan Reviewed By: chad CC: shanemhansen, aran Maniphest Tasks: T2821 Differential Revision: https://secure.phabricator.com/D5422
1 parent 178d732 commit cb84cb1

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed
 

‎src/applications/people/controller/PhabricatorEmailVerificationController.php

+4
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@ public function willProcessRequest(array $data) {
99
$this->code = $data['code'];
1010
}
1111

12+
public function shouldRequireAdmin() {
13+
return false;
14+
}
15+
1216
public function shouldRequireEmailVerification() {
1317
// Since users need to be able to hit this endpoint in order to verify
1418
// email, we can't ever require email verification here.

0 commit comments

Comments
 (0)
Failed to load comments.