Last modified: 2013-04-17 12:37:21 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 T48680, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 46680 - Firefox pre-fetches some special pages when they are returnto
Firefox pre-fetches some special pages when they are returnto
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.21.x
All All
: Unprioritized normal (vote)
: ---
Assigned To: Matthew Flaschen
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-03-29 02:40 UTC by Matthew Flaschen
Modified: 2013-04-17 12:37 UTC (History)
2 users (show)

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


Attachments

Description Matthew Flaschen 2013-03-29 02:40:40 UTC
Since bug 16707 resolved, MediaWiki sets rel="next" to the returnto.  Although the stated reason is for UX (e.g. it mentions Opera can show some kind of Next link/button), it has other impacts.

Firefox (and possibly other browsers) use this as a pre-fetching hint.  For regular pages, this can be fine.  However, some special pages (I know of ConfirmEmail and InvalidateEmail, but there may be others) take an action on a regular GET request, even though this isn't generally best practice.

Firefox will not pre-fetch if there is a query string (see https://bugzilla.mozilla.org/show_bug.cgi?id=175418), so my proposed fix adds this to the rel=next URL, only for special pages that don't already have a query string.  It does not affect the actual displayed returnto link (<a> element).
Comment 1 Matthew Flaschen 2013-03-29 02:49:11 UTC
Proposed fix: https://gerrit.wikimedia.org/r/#/c/56565/
Comment 2 Matthew Flaschen 2013-03-29 19:00:01 UTC
The consensus there was that rel="next" is not really correct here, because e.g. the prev of the page you're going to was not the, e.g. login page, and two pages isn't really a sequence.

So I'm now proposing to just remove the rel="next":

https://gerrit.wikimedia.org/r/#/c/56624/
Comment 3 Tyler Romeo 2013-03-29 20:32:36 UTC
(In reply to comment #2)
> The consensus there was that rel="next" is not really correct here, because
> e.g. the prev of the page you're going to was not the, e.g. login page, and
> two
> pages isn't really a sequence.
> 
> So I'm now proposing to just remove the rel="next":
> 
> https://gerrit.wikimedia.org/r/#/c/56624/

Merged.
Comment 4 Gerrit Notification Bot 2013-04-17 12:37:21 UTC
Related URL: https://gerrit.wikimedia.org/r/59620 (Gerrit Change I6a51e3ee07fe7622b9c708c78563795d7a1118fc)

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


Navigation
Links