Page MenuHomePhabricator

Non-latin links shouldn't be encoded
Open, LowestPublic

Event Timeline

Base assigned this task to Petrb.
Base raised the priority of this task from to Needs Triage.
Base updated the task description. (Show Details)
Base added a project: WM-Bot.
Base added a subscriber: Base.
Poyekhali triaged this task as Medium priority.Apr 13 2016, 2:53 AM
Poyekhali added a subscriber: Poyekhali.

Seems to be reasonable for me, because making it encoded will make the link longer. Also, IRC messages are limited in size.

Petrb lowered the priority of this task from Medium to Lowest.Dec 23 2018, 5:30 PM


Current code for encoding is

link = HttpUtility.UrlEncode(link).Replace("%2f", "/")
    .Replace("%3a", ":")
    .Replace("%23", "#")
    .Replace("+", "_");

Any idea how to make it not encode these characters you mean?

Petrb removed Petrb as the assignee of this task.Mar 25 2020, 12:01 AM
Petrb added a subscriber: Petrb.