Linky filter in angular-sanitize is ignoring last parenthesis in URL ending with a parenthesis

[jd2791]

Using angular-santize.js 1.2.9 and angular.js 1.2.9, the linky filter ignores the final parenthesis in a given URL that has a parenthesis as the last character. This appears to happen in all browsers. An example can be found here: http://jsfiddle.net/Uqg9D/1/

[petebacondarwin]

Here also is a Plnkr with 1.2.16 http://plnkr.co/edit/oHUhXl6xu5ZwNyNTK9oz?p=preview
and 1.3.0-beta.6 http://plnkr.co/edit/y6obJAqPFo0bNoBZj4KN?p=preview

[petebacondarwin]

Clearly this is not correct but to fix this is problematic.

The current regex in linky is here: https://github.com/angular/angular.js/blob/master/src/ngSanitize/filter/linky.js#L106

/((ftp|https?):\/\/|(mailto:)?[A-Za-z0-9._%+-]+@)\S*[^\s.;,(){}<>]/

This is not including any trailing punctuation, presumable so that URLs can be found that end sentences, and are enclosed in brackets, etc.

But URLs, according to the spec, can end with any "xalpha":

  xalpha                  alpha | digit | safe | extra | escape

where notably,

  safe                    $ | - | _ | @ | . | &  | + | -                        
  extra                  ! | * |  " |  ' | ( | )  | , 

So URLs can indeed end with full stops and parenthesis.

Allowing them will break people's expectation of how link should work in free form text but disallowing them potentially fails to match valid URLs.

Any thoughts?

[petebacondarwin]

The markdown solution is to require "automatic" URLs to be wrapped in < angle brackets >