Last modified: 2013-11-22 19:41:27 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 T59289, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 57289 - Invalid "Change Password" (returnto) page shown after successful temporary password login & password change
Invalid "Change Password" (returnto) page shown after successful temporary pa...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
User login and signup (Other open bugs)
1.23.0
All All
: High major (vote)
: ---
Assigned To: Nobody - You can work on this!
: code-update-regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-20 09:20 UTC by T. Gries
Modified: 2013-11-22 19:41 UTC (History)
5 users (show)

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


Attachments

Description T. Gries 2013-11-20 09:20:03 UTC
* 1.23wmf3 (e2e9b85)

Scenario:

+ en.wikipedia.org as of 2013-11-20
+ go to "Reset Password" page and trigger the I-forgot-my-passwort e-mail-password mail
+ come back to the login page
+ enter the temporary password
+ you are now correctly asked to change your password (=mandatory password change after login with temporary password)
+ after a successful password change you will see

Bug:
===

page title after successful password change is (still): 

"Change Password"

URL is: 

"https://en.wikipedia.org/w/index.php?title=Special:ChangePassword&returnto=&returntoquery=&fromhttp=1"

with unsuited information on it:

"You must be logged in to access this page directly."

This is striclty reproducible.
Comment 1 T. Gries 2013-11-20 10:55:44 UTC
perhaps the fix of https://bugzilla.wikimedia.org/show_bug.cgi?id=57065 solves this.
Comment 2 T. Gries 2013-11-20 10:58:10 UTC
(In reply to comment #1)
> perhaps the fix of https://bugzilla.wikimedia.org/show_bug.cgi?id=57065
> solves
> this.
^wrong

I meant:

See 
https://bugzilla.wikimedia.org/show_bug.cgi?id=57098 SpecialPasswordReset when called does not obey an optional returnto parameter 

Perhaps the fix of 57098 also solves the present 57289.
Comment 3 Andre Klapper 2013-11-21 21:17:19 UTC
Thanks for taking the time to report this!

Confirming:

- logged out
- went to https://en.wikipedia.org/wiki/Special:PasswordReset
- enter username
- Get "A password reset email has been sent."
- Check mail
- Copy temp password
- In browser, click "Log in" in upper corner which now links to https://en.wikipedia.org/w/index.php?title=Special:UserLogin&returnto=Special%3APasswordReset
- Enter username and temp password
- Get "Change password - You logged in with a temporary emailed code. To finish logging in, you must set a new password here:"

Result:
- "Change password - You must be logged in to access this page directly."
Needed to log in once again. Meh.
Comment 4 T. Gries 2013-11-21 22:04:11 UTC
> Needed to log in once again. Meh.

just to say it very kindly again in case it has been overlooked:
> __perhaps_(!)_ the fix of https://bugzilla.wikimedia.org/show_bug.cgi?id=57098 solves this, too.
Comment 5 Chris Steipp 2013-11-22 03:51:39 UTC
Is this still happening? I'm not able to reproduce it. Just to confirm,

* I enter my username and temp password, click login
* I get the change password form. I'm not logged in. Old password is pre-filled.
* I enter in a new password (twice), and click submit.
* I'm taken back to Main_Page, and I'm logged in.

Am I missing a step?
Comment 6 T. Gries 2013-11-22 06:37:15 UTC
(In reply to comment #5)
> Is this still happening? I'm not able to reproduce it. Just to confirm,
> 
> * I enter my username and temp password, click login
> * I get the change password form. I'm not logged in. Old password is
> pre-filled.
> * I enter in a new password (twice), and click submit.
> * I'm taken back to Main_Page, and I'm logged in.
> 
> Am I missing a step?

Yes, certainly.
I still can reproduce my findings as in the first comment, and as confirmed by André in https://bugzilla.wikimedia.org/show_bug.cgi?id=57289#c3


Please reproduce step-by-step:


Scenario:

+ en.wikipedia.org as of 2013-11-22
+ go to "Reset Password" page and trigger the I-forgot-my-passwort
e-mail-password mail
+ come back to the login page
+ enter the temporary password
+ you are now correctly asked to change your password (=mandatory password
change after login with temporary password)
+ after a successful password change you will see

"Change Password" (=page title)
"You must be logged in to access this page directly."

URL is: 
"https://en.wikipedia.org/w/index.php?title=Special:ChangePassword&returnto=&returntoquery=&fromhttp=1"

This is *strictly* reproducible. Raising severity to "major" because it relates to a password issue, which could point to critical bug.
Comment 7 T. Gries 2013-11-22 06:41:03 UTC
screenshot of the page after successfully having entered the temp.password and 2x the new password:

+ http://i.imgur.com/IDNp8W4.png
Comment 8 Gerrit Notification Bot 2013-11-22 09:37:36 UTC
Change 96970 had a related patch set uploaded by IAlex:
Fix login with temporary password with $wgSecureLogin = true

https://gerrit.wikimedia.org/r/96970
Comment 9 Tyler Romeo 2013-11-22 12:26:00 UTC
For the record, the cause of this bug is from Ia0a61e98fbff7 ( https://gerrit.wikimedia.org/r/93425), which introduced WebRequest::getProtocol as an instance method rather than the previous practice of always using WebRequest::detectProtocol, which is static.
Comment 10 Gerrit Notification Bot 2013-11-22 14:37:07 UTC
Change 96994 had a related patch set uploaded by MarkAHershberger:
Fix login with temporary password with $wgSecureLogin = true

https://gerrit.wikimedia.org/r/96994
Comment 11 Gerrit Notification Bot 2013-11-22 16:09:09 UTC
Change 96970 merged by MarkAHershberger:
Fix login with temporary password with $wgSecureLogin = true

https://gerrit.wikimedia.org/r/96970
Comment 12 Gerrit Notification Bot 2013-11-22 16:15:27 UTC
Change 96994 merged by MarkAHershberger:
Fix login with temporary password with $wgSecureLogin = true

https://gerrit.wikimedia.org/r/96994
Comment 13 T. Gries 2013-11-22 19:02:38 UTC
In www.mediawiki,org : Not solved


After changing the password, see http://i.imgur.com/7ADHPHH.png

URL:

https://www.mediawiki.org/w/index.php?title=Special:ChangePassword&returnto=&returntoquery=&fromhttp=1

Problem is not solved, in my view.
Comment 14 Tyler Romeo 2013-11-22 19:37:42 UTC
That's because the fix has not been deployed yet. However, the actual bug in question has been resolved in master.
Comment 15 T. Gries 2013-11-22 19:41:27 UTC
Tyler: okay. I understand this from a discussion in the chat some minutes ago. ty

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


Navigation
Links