Page MenuHomePhabricator

"Permanently disable mobile site" doesn't work due to an encoding problem
Closed, ResolvedPublic

Description

  1. http://zh.m.wikipedia.org/, click "Permanently disable mobile site"
  2. Click "Disable"

Then the page goes to http://zh.wikipedia.org/w/mobileRedirect.php?to=http%253A%252F%252Fzh.wikipedia.org%252Fwiki%252FPortal%253A%25E7%25A7%25BB%25E5%258A%25A8%25E7%2589%2588%25E9%25A6%2596%25E9%25A1%25B5&expires_in_days=3650. And then whoops, "Invalid target". Seems the URL is some kinds of double-encoded, http:// -> http%3A%2F%2F -> http%253A%252F%252F.

Reproduced on every mobile sites.


Version: .5
Severity: major

Details

Reference
bz23229

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:08 PM
bzimport set Reference to bz23229.

hcatlin wrote:

Fixed this issue. Thanks for the great bug report. I was able to track it down easily.