@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
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
+2
-2
+2
-2
src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php
+2
-2
src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php
···
10
10
public function handleRequest(AphrontRequest $request) {
11
11
$viewer = $this->getViewer();
12
12
$id = $request->getURIData('id');
13
-
$link_type = $request->getURIData('key');
14
-
$key = $request->getURIData('type');
13
+
$link_type = $request->getURIData('type');
14
+
$key = $request->getURIData('key');
15
15
$email_id = $request->getURIData('emailID');
16
16
17
17
if ($request->getUser()->isLoggedIn()) {