Last modified: 2014-02-12 23:47:37 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 T51647, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49647 - in Experimental, some accents in urls have recursive problems
in Experimental, some accents in urls have recursive problems
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
beta (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Jon
:
: 49804 53455 (view as bug list)
Depends on:
Blocks: 49775
  Show dependency treegraph
 
Reported: 2013-06-16 19:19 UTC by Sage Ross
Modified: 2014-02-12 23:47 UTC (History)
9 users (show)

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


Attachments

Description Sage Ross 2013-06-16 19:19:32 UTC
With Experimental settings on, I go to the following url:
https://en.m.wikipedia.org/wiki/Cr%C3%AApe

This gets rewritten as:
https://en.m.wikipedia.org/wiki/Cr%C3%83%C2%AApe

Upon each refresh, the size of the url increases until I end up with:
https://en.m.wikipedia.org/wiki/Cr%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%83%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%83%C3%83%C2%82%C3%82%C2%82%C3%83%C2%83%C3%82%C2%82%C3%83%C2%82%C3%82%C2%AApe

On the stock Browswer browser for Android, the page refreshes with each url on its own, meaning that the target article cannot even be viewed. On desktop, the urls only get rewritten and expanded upon each manual page refresh.
Comment 1 Jon 2013-06-17 19:31:55 UTC
Very strange. I also notice searching from Crepe on the main page mangles the URL as well.

This could be an issue in MobileFrontend or the History.js library we use.

I notice calling
History.replaceState( null, mw.config.get( 'wgTitle' ), decodeURIComponent( window.location.pathname ) );

instead of
History.replaceState( null, mw.config.get( 'wgTitle' ) )
fixes the reported problem (although pages loaded via the api also have this problem)

arggg encoding issues.
Comment 2 Jon 2013-06-17 22:33:51 UTC
Seems to be an upstream bug:
https://github.com/browserstate/history.js/issues/299
Comment 3 Jon 2013-06-19 16:54:22 UTC
*** Bug 49804 has been marked as a duplicate of this bug. ***
Comment 4 Jon 2013-06-19 16:55:13 UTC
In Dolphin this also seems to cause a redirect.

Going to "Ä" redirect to "Ã", then to "ÂÃ" etc.. until it ends up at "Bad
title". This only occurs in dolphin browser on my phone, and not in chrome, nor
in firefox.
Comment 5 Jon 2013-06-19 16:56:35 UTC
(This can also happen on Chrome desktop)
Comment 6 Gerrit Notification Bot 2013-06-28 00:45:32 UTC
Change 70953 had a related patch set uploaded by Jdlrobson:
Bug 49647: Take away History.js dependency

https://gerrit.wikimedia.org/r/70953
Comment 7 Jon 2013-08-28 07:14:11 UTC
*** Bug 53455 has been marked as a duplicate of this bug. ***
Comment 8 Gerrit Notification Bot 2013-09-06 20:52:33 UTC
Change 70953 merged by jenkins-bot:
Bug 49647: Take away History.js dependency

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

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


Navigation
Links