fix(linky): handle quotes around email addresses

btford · btford · commit a9d227120dc2 · 2014-08-20T13:37:45.000-07:00
Closes #8520
diff --git a/src/ngSanitize/filter/linky.js b/src/ngSanitize/filter/linky.js
@@ -104,7 +104,7 @@
  */
 angular.module('ngSanitize').filter('linky', ['$sanitize', function($sanitize) {
   var LINKY_URL_REGEXP =
-        /((ftp|https?):\/\/|(mailto:)?[A-Za-z0-9._%+-]+@)\S*[^\s.;,(){}<>]/,
+        /((ftp|https?):\/\/|(mailto:)?[A-Za-z0-9._%+-]+@)\S*[^\s.;,(){}<>"]/,
       MAILTO_REGEXP = /^mailto:/;
 
   return function(text, target) {
diff --git a/test/ngSanitize/filter/linkySpec.js b/test/ngSanitize/filter/linkySpec.js
@@ -25,6 +25,8 @@ describe('linky', function() {
                     toEqual('<a href="mailto:me@example.com">me@example.com</a>');
     expect(linky("send email to me@example.com, but")).
       toEqual('send email to <a href="mailto:me@example.com">me@example.com</a>, but');
+    expect(linky("my email is \"me@example.com\"")).
+      toEqual('my email is &#34;<a href="mailto:me@example.com">me@example.com</a>&#34;');
   });
 
   it('should handle target:', function() {

-Original file line number
+Diff line change
                     toEqual('<a href="mailto:[email protected]">[email protected]</a>');
     expect(linky("send email to [email protected], but")).
       toEqual('send email to <a href="mailto:[email protected]">[email protected]</a>, but');
 +    expect(linky("my email is \"[email protected]\"")).
 +      toEqual('my email is &#34;<a href="mailto:[email protected]">[email protected]</a>&#34;');
   });
   it('should handle target:', function() {