fix($compile): whitelist file:// in url sanitization

angular · Feb 25, 2013 · 7b236b2 · 7b236b2
1 parent c36933d
commit 7b236b2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/ng/compile.js b/src/ng/compile.js
@@ -156,7 +156,7 @@ function $CompileProvider($provide) {
       COMMENT_DIRECTIVE_REGEXP = /^\s*directive\:\s*([\d\w\-_]+)\s+(.*)$/,
       CLASS_DIRECTIVE_REGEXP = /(([\d\w\-_]+)(?:\:([^;]+))?;?)/,
       MULTI_ROOT_TEMPLATE_ERROR = 'Template must have exactly one root element. was: ',
-      urlSanitizationWhitelist = /^\s*(https?|ftp|mailto):/;
+      urlSanitizationWhitelist = /^\s*(https?|ftp|mailto|file):/;
 
 
   /**

diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
@@ -2479,6 +2479,10 @@ describe('$compile', function() {
       $rootScope.testUrl = "mailto:foo@bar.com";
       $rootScope.$apply();
       expect(element.attr('href')).toBe('mailto:foo@bar.com');
+
+      $rootScope.testUrl = "file:///foo/bar.html";
+      $rootScope.$apply();
+      expect(element.attr('href')).toBe('file:///foo/bar.html');
     }));