// Remove ?markasread=XYZ from the URL
var uri = new mw.Uri();
if ( uri.query.markasread !== undefined ) {
        delete uri.query.markasread;
        window.history.replaceState( null, document.title, uri );
}

So I strongly suspect this is a bug in mw.Uri.

Catrope edited projects, added Collaboration-Team-Triage (Collab-Team-This-Quarter); removed Collaboration-Team-Triage.Jan 9 2018, 7:31 PM

Catrope moved this task from Untriaged to Ready for Pickup on the Collaboration-Team-Triage (Collab-Team-This-Quarter) board.

Change 405377 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/core@master] mediawiki.Uri: Don't double-escape fragments

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

gerritbot added a project: Patch-For-Review.Jan 19 2018, 11:17 PM

Catrope moved this task from Ready for Pickup to Needs Review on the Collaboration-Team-Triage (Collab-Team-This-Quarter) board.Jan 19 2018, 11:17 PM

Change 405377 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.Uri: Don't double-escape fragments

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

ReleaseTaggerBot added a project: MW-1.31-release-notes (WMF-deploy-2018-02-06 (1.31.0-wmf.20)).Jan 20 2018, 4:00 AM

SBisson moved this task from Needs Review to QA Review on the Collaboration-Team-Triage (Collab-Team-This-Quarter) board.Feb 1 2018, 1:58 PM

Checked the fix in testwiki (wmf.20) - no double-encoding in URI when markasread is removed.

Etonkovidova closed this task as Resolved.Feb 7 2018, 11:23 PM

Echo double-encodes URI 'fragment' part when removing the 'markasread' parameterClosed, ResolvedPublicActions

Description

Details

Event Timeline

Echo double-encodes URI 'fragment' part when removing the 'markasread' parameter
Closed, ResolvedPublic
Actions