Last modified: 2013-10-02 19:27:30 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T55651, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53651 - Redirect to HTTPS for account creation
Redirect to HTTPS for account creation
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
SSL related (Other open bugs)
wmf-deployment
All All
: Highest normal with 1 vote (vote)
: ---
Assigned To: Chris Steipp
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-01 11:26 UTC by Dan Wolff
Modified: 2013-10-02 19:27 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Dan Wolff 2013-09-01 11:26:03 UTC
Now the HTTP login page redirects to the HTTPS version, but for some reason creating an account is exempt.

Is there any reason to allow the login details to be sent in cleartext even once?

What I expect: [1] should redirect to [2]

[1] http://en.wiktionary.org/w/index.php?title=Special:UserLogin&returnto=Wiktionary%3AMain+Page&type=signup

[2] https://en.wiktionary.org/w/index.php?title=Special:UserLogin&returnto=Wiktionary%3AMain+Page&type=signup
Comment 1 chmarkine 2013-09-03 21:41:03 UTC
Not only for Wiktionary, there are lots of signup pages which should redirect to https. For instance,

http://en.wikipedia.org/w/index.php?title=Special:UserLogin&returnto=Main+Page

should redirect to

https://en.wikipedia.org/w/index.php?title=Special:UserLogin&returnto=Main+Page

But currently it doesn't redirect.
Comment 2 chmarkine 2013-09-03 21:44:45 UTC
(In reply to comment #1)
Sorry, I meant:

http://en.wikipedia.org/w/index.php?title=Special:UserLogin&returnto=Main+Page&type=signup

and 

https://en.wikipedia.org/w/index.php?title=Special:UserLogin&returnto=Main+Page&type=signup

(In the original post, the link points to the login page rather than the signup page.)
Comment 3 Greg Grossmeier 2013-09-09 16:47:12 UTC
Confirmed on enwiki. Re-prioritizing.
Comment 4 Chris Steipp 2013-09-09 17:32:36 UTC
I have vague memories of doing that for a reason, but I can't seem to find it right now. Git blame points to Tyler for the $this->mType !== 'signup' in the code. Tyler, do you remember why signup was excluded there?
Comment 5 Tyler Romeo 2013-09-10 04:20:46 UTC
I can't seem to recall why I put that condition there. I don't think there was a reason. The commit this condition was added on was way back in 565014a8, when I originally fixed $wgSecureLogin functionality (because before that the feature was completely broken and didn't work). It should be safe to just take that out of the if() statement.
Comment 6 Andre Klapper 2013-10-02 14:58:22 UTC
csteipp: Any news here, after Tyler's reply?
Comment 7 Tyler Romeo 2013-10-02 19:27:30 UTC
Woops

https://gerrit.wikimedia.org/r/86314
Merged

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links