diff options
| author | Anna Vester | 2012-11-10 14:37:17 -0600 |
|---|---|---|
| committer | Igor Minar | 2012-11-11 10:31:27 +0100 |
| commit | 04450c48dfea065e1c9e4ab8adad94993ed1b037 (patch) | |
| tree | a4a92f8a3f1ec5646248beccb50da3b90411ff1b | |
| parent | 8650843603be6f6f17aa3f90fb9991420669cb53 (diff) | |
| download | angular.js-04450c48dfea065e1c9e4ab8adad94993ed1b037.tar.bz2 | |
feat($sanitize): support telephone links
Per http://www.ietf.org/rfc/rfc3966.txt support tel: links
| -rw-r--r-- | src/ngSanitize/sanitize.js | 2 | ||||
| -rw-r--r-- | test/ngSanitize/sanitizeSpec.js | 5 |
2 files changed, 4 insertions, 3 deletions
diff --git a/src/ngSanitize/sanitize.js b/src/ngSanitize/sanitize.js index 677b11d3..04eafa0d 100644 --- a/src/ngSanitize/sanitize.js +++ b/src/ngSanitize/sanitize.js @@ -123,7 +123,7 @@ var START_TAG_REGEXP = /^<\s*([\w:-]+)((?:\s+[\w:-]+(?:\s*=\s*(?:(?:"[^"]*")|(?: BEGING_END_TAGE_REGEXP = /^<\s*\//, COMMENT_REGEXP = /<!--(.*?)-->/g, CDATA_REGEXP = /<!\[CDATA\[(.*?)]]>/g, - URI_REGEXP = /^((ftp|https?):\/\/|mailto:|#)/, + URI_REGEXP = /^((ftp|https?):\/\/|mailto:|tel:|#)/, NON_ALPHANUMERIC_REGEXP = /([^\#-~| |!])/g; // Match everything outside of normal chars and " (quote character) diff --git a/test/ngSanitize/sanitizeSpec.js b/test/ngSanitize/sanitizeSpec.js index b4fd8a2a..150072d9 100644 --- a/test/ngSanitize/sanitizeSpec.js +++ b/test/ngSanitize/sanitizeSpec.js @@ -188,7 +188,7 @@ describe('HTML', function() { expect(html).toEqual('<div>'); }); - describe('explicitly dissallow', function() { + describe('explicitly disallow', function() { it('should not allow attributes', function() { writer.start('div', {id:'a', name:'a', style:'a'}); expect(html).toEqual('<div>'); @@ -230,10 +230,11 @@ describe('HTML', function() { expect(isUri('https://abc')).toBeTruthy(); expect(isUri('ftp://abc')).toBeTruthy(); expect(isUri('mailto:me@example.com')).toBeTruthy(); + expect(isUri('tel:123-123-1234')).toBeTruthy(); expect(isUri('#anchor')).toBeTruthy(); }); - it('should not be UIR', function() { + it('should not be URI', function() { expect(isUri('')).toBeFalsy(); expect(isUri('javascript:alert')).toBeFalsy(); }); |