Defect #1456
Invalid html generated by wiki formatter
| Status: | Closed | Start: | 2008-06-15 | |
| Priority: | Normal | Due date: | ||
| Assigned to: | - | % Done: | 0% |
|
| Category: | Wiki | |||
| Target version: | 0.8 | |||
| Affected version: | 0.7.2 |
Resolution: | Fixed |
|
Description
Links generated by the wiki are invalid when they contain something that could be an email. For example, this :
http://foo@www.bar.com
is transformed to
<a class=\"external\" href=\"http://<a href=\"mailto:www.foo@bar.com\" class=\"email\">www.foo@bar.com</a>\">http://<a href=\"mailto:www.foo@bar.com\" class=\"email\">www.foo@bar.com</a></a>
and rendered as this http://foo@www.bar.com in a web browser.
I observed this problem by using hg fetch that generates commit messages like :
Automated merge with https://login@hg.foo.com/project.
Here is a patch that solves the problem and a unit test. The code come from http://dev.rubyonrails.org/changeset/7516. The only limitation of this solution is in that case :
http://foo@bar.com foo@bar.com
where only the first link is detected. I did not found the magic regexp. Maybe this could be solved with some lookbehind but the syntax is not supported on ruby 1.8.
Associated revisions
Fixed: urls containing @ are parsed as email adress by the wiki formatter (#1456).