revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-10 00:42:41 +01:00
Code Issues Releases Wiki Activity

Fix next_uri

Browse source 
Summary: we were using the "path" as the next_uri and that drops some delicious get parameters

Test Plan: see T1140; basically re-ran the steps listed there and they passed!

Reviewers: epriestley, njhartwell

Reviewed By: epriestley

CC: aran, Koolvin

Maniphest Tasks: T1140, T1009

Differential Revision: https://secure.phabricator.com/D2299
This commit is contained in:
Bob Trahan 2012-04-22 20:39:54 +02:00
parent c53dd9968b
commit 8069694640
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/applications/auth/controller/login/PhabricatorLoginController.php
View file

@ -64,9 +64,11 @@ final class PhabricatorLoginController
)); ));
} }
$next_uri = $this->getRequest()->getPath(); $next_uri_path = $this->getRequest()->getPath();
if ($next_uri == '/login/') { if ($next_uri_path == '/login/') {
$next_uri = '/'; $next_uri = '/';
} else {
$next_uri = $this->getRequest()->getRequestURI();
} }
if (!$request->isFormPost()) { if (!$request->isFormPost()) {