Last modified: 2013-09-04 11:53:10 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 T34709, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32709 - Private Wiki users are always taken to Special:Badtitle on successful login
Private Wiki users are always taken to Special:Badtitle on successful login
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
User login and signup (Other open bugs)
1.17.x
All All
: High major with 2 votes (vote)
: 1.18.x release
Assigned To: Antoine "hashar" Musso (WMF)
: platformeng
Depends on:
Blocks: 32711
  Show dependency treegraph
 
Reported: 2011-11-29 16:31 UTC by Phil Driscoll
Modified: 2013-09-04 11:53 UTC (History)
10 users (show)

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


Attachments

Description Phil Driscoll 2011-11-29 16:31:56 UTC
When visiting a private wiki, users are shown the message
'You must log in to view other pages'.
Because of the recent changes in 1.17.1 that hide page titles from users who don't have permission to see them,  the log in link now has a returnto of Special:BadTitle.

The net result is that when a legitimate user logs in, they are taken to the Badtitle error page, rather than to the Mainpage.
Comment 1 Brion Vibber 2011-11-30 00:45:09 UTC
I get this on https://office.wikimedia.org/ as well, so the same bug hit 1.18. :P
Comment 2 Olivier Finlay Beaton 2011-11-30 14:29:55 UTC
I am not able to reproduce this on 1.18.0 release or trunk 2011-11-29. However it occurs every single time on 1.17.1, all stock installs.
Comment 3 Brion Vibber 2011-12-02 21:12:56 UTC
Special:BadTitle gets passed as the 'returnto' parameter for the login link that's displayed in the page body content when unable to view a page due to permissions.

The login link in the personal bar at top of the screen still has the original page title.
Comment 4 Rob Lanphier 2011-12-02 21:17:18 UTC
Almost certainly introduced in r104505.  Just need to fix in a way that doesn't reintroduce bug 32276
Comment 5 Brion Vibber 2011-12-02 21:26:13 UTC
Notes from triage:

the 'You must log in to view other pages' message comes from OutputPage::showPermissionsErrorPage().

r104505 on trunk has a tweak in that function to grab the original title from $wgRequest's 'title' parameter.

The backports to 1.17, 1.18, and 1.18wmf1 don't appear to include this part, so presumably it's still pulling from the variable that's been replaced with Special:Badtitle.
Comment 6 Florian D. 2011-12-06 14:18:35 UTC
As for me, I *do* reproduce it on 1.18.0 release.
Comment 7 Tim Starling 2011-12-13 00:00:24 UTC
(In reply to comment #5)
> Notes from triage:
> 
> the 'You must log in to view other pages' message comes from
> OutputPage::showPermissionsErrorPage().
> 
> r104505 on trunk has a tweak in that function to grab the original title from
> $wgRequest's 'title' parameter.
> 
> The backports to 1.17, 1.18, and 1.18wmf1 don't appear to include this part, so
> presumably it's still pulling from the variable that's been replaced with
> Special:Badtitle.

Reassigning to Antoine since I think he did those backports.
Comment 8 Antoine "hashar" Musso (WMF) 2011-12-13 21:37:39 UTC
I am pretty sure we had some code to avoid returning to a badtitle and return to main_page instead. I must have skipped it while doing the backports.

Will poke this bug ASAP.
Comment 9 Antoine "hashar" Musso (WMF) 2011-12-14 09:42:16 UTC
Fixed in:
- REL1_17 with r106170
- REL1_18 with r106171

Pending code review.
Comment 10 Antoine "hashar" Musso (WMF) 2011-12-14 21:11:47 UTC
Merged in production with r106206 and deployed by Roan a few minutes after.
The issue should have disappeared from office wiki for example.
Comment 11 Brion Vibber 2011-12-14 21:23:06 UTC
Confirmed fixed!
Comment 12 p858snake 2011-12-14 22:49:09 UTC
*** Bug 33130 has been marked as a duplicate of this bug. ***

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


Navigation
Links