https://bugzilla.wikimedia.org/show_bug.cgi?id=21261

--- Comment #8 from Platonides <[email protected]> 2011-11-26 21:47:53 UTC 
---
The patch by Peter looks the way to go, but I'm afraid adding brackets to
EXT_LINK_URL_CLASS would allow too much recursion. And that wouldn't stop at
the ] of a url.
The change to the sanitizer is needed to avoid converting
http://[2404:130:0:1000::187:2]/index.php to
http://%5B2404:130:0:1000::187:2%5D/index.php but although that doesn't break
any parser test, I suspect there was a reason they were added as control
characters in r14930, but I don't know which one (avoid treating the content
like? that would no longer hold...).

Note that the change to mExtLinkBracketedRegex won't apply, but it's now
constructed from EXT_LINK_URL_CLASS so no problem there.

I added a parser test in r104314.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to