From b4b5d60f77fbcac420d773f6fa9bef1c6eebdfa9 Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 3 Aug 2015 08:01:43 -0700 Subject: [PATCH] Fix 'key'/'type' swap in email reset / one-time-login controller Summary: Fixes T9046. These got swapped around during refactoring. Test Plan: - Used `bin/auth recover` prior to patch (failed). - Used `bin/auth recover` after patch (worked). Reviewers: joshuaspence, chad Reviewed By: chad Subscribers: epriestley Maniphest Tasks: T9046 Differential Revision: https://secure.phabricator.com/D13778 --- .../auth/controller/PhabricatorAuthOneTimeLoginController.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php b/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php index 91f3d6a984..6dfa49d860 100644 --- a/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php +++ b/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php @@ -10,8 +10,8 @@ final class PhabricatorAuthOneTimeLoginController public function handleRequest(AphrontRequest $request) { $viewer = $this->getViewer(); $id = $request->getURIData('id'); - $link_type = $request->getURIData('key'); - $key = $request->getURIData('type'); + $link_type = $request->getURIData('type'); + $key = $request->getURIData('key'); $email_id = $request->getURIData('emailID'); if ($request->getUser()->isLoggedIn()) { -- 2.51.2