Ticket #451 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

URL matching misses trailing slashes (and ! and .)

Reported by: h3h Assigned to: timothy
Priority: normal Component: Colloquy (GUI)
Version: 2.0 (2D16) Severity: minor
Keywords: Cc:

Description

URLs in plain text such as http://test.com/ and http://test.com/test/ should be made into links in their entirety, including the trailing slashes. Right now the trailing slashes are not included in the created hyperlink.

Change History

01/10/06 13:48:32 changed by Joost de Valk (AlthA)

Have the same problem, it's kind of annoying since it takes a redirect on a lot of pages...

08/21/06 17:54:43 changed by florent.bruneau_clqy@m4x.org

I find Colloquy's URL catcher very weak. I don't know how you catch your URLs, but I'm used to using the following regexp : ((?:https?|ftp|news)://(?:&|\.*,*[a-z@0-9~%$£µ&i#\-+=_/\?])*)

I made a lot of tests with this regexp. It catches correctly all the URLs Colloquy failed to recognize. And because it has been designed for an nntp client, this regexp can safely be used in order to catch URLs in a real time discussion.

I hope you will find this useful. Greetings

08/29/06 08:09:53 changed by akempgen

  • summary changed from URL matching misses trailing slashes to URL matching misses trailing slashes (and ! and .).

from #849: ! and . are missing too, not sure if they are allowed in urls, though

11/11/06 23:59:34 changed by timothy

  • status changed from new to closed.
  • resolution set to fixed.

Fixed in [3383].