Cicalese has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/372760 )

Change subject: Encode url properly
......................................................................


Encode url properly

Bug: T173627
Change-Id: I165bbc3f56394fd135d21c982695cfb21ba37b8c
---
M PluggableAuthHooks.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Cicalese: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/PluggableAuthHooks.php b/PluggableAuthHooks.php
index 325dc6f..f98ba72 100644
--- a/PluggableAuthHooks.php
+++ b/PluggableAuthHooks.php
@@ -157,7 +157,7 @@
                $oldTitle = $title;
                $title = Title::newFromText( "UserLogin", NS_SPECIAL );
                $out->redirect( $title->getFullURL( [
-                       'returnto' => $oldTitle,
+                       'returnto' => urlencode( $oldTitle ),
                        'returntoquery' => $request->getRawQueryString()
                ] ) );
        }

-- 
To view, visit https://gerrit.wikimedia.org/r/372760
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I165bbc3f56394fd135d21c982695cfb21ba37b8c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PluggableAuth
Gerrit-Branch: master
Gerrit-Owner: MarkAHershberger <[email protected]>
Gerrit-Reviewer: Cicalese <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to