Jdlrobson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/277918

Change subject: Remove source of a possible redirect loop
......................................................................

Remove source of a possible redirect loop

Rather than handle this in MobileFrontend use the same mechanism
core and desktop use.

Change-Id: I1368948a4cb8e43eaf9c6608506c9a034a690adb
---
M includes/MobileFrontend.hooks.php
1 file changed, 0 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/18/277918/1

diff --git a/includes/MobileFrontend.hooks.php 
b/includes/MobileFrontend.hooks.php
index c66cc34..278bb6c 100644
--- a/includes/MobileFrontend.hooks.php
+++ b/includes/MobileFrontend.hooks.php
@@ -630,22 +630,6 @@
                        } elseif ( $name === 'Userlogin' ) {
                                $out->addModuleStyles( 
'skins.minerva.special.userlogin.styles' );
                                $out->addModules( 
'mobile.special.userlogin.scripts' );
-                               // make sure we're on https if we're supposed 
to be and currently aren't.
-                               // most of this is lifted from https redirect 
code in SpecialUserlogin::execute()
-                               // also, checking for 'https' in $wgServer is a 
little funky, but this is what
-                               // is done on the WMF cluster (see config in 
CommonSettings.php)
-                               if ( $secureLogin && 
WebRequest::detectProtocol() != 'https' ) {
-                                       // get the https url and redirect
-                                       $query = 
$special->getContext()->getRequest()->getQueryValues();
-                                       if ( isset( $query['title'] ) )  {
-                                               unset( $query['title'] );
-                                       }
-                                       $url = $mobileContext->getMobileUrl(
-                                               
$special->getFullTitle()->getFullURL( $query ),
-                                               true
-                                       );
-                                       
$special->getContext()->getOutput()->redirect( $url );
-                               }
                        }
                }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1368948a4cb8e43eaf9c6608506c9a034a690adb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: wmf/1.27.0-wmf.17
Gerrit-Owner: Jdlrobson <[email protected]>

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

Reply via email to