diff --git a/src/Angular.js b/src/Angular.js index eaaf2f37bc75..c515ff93d80f 100644 --- a/src/Angular.js +++ b/src/Angular.js @@ -921,12 +921,23 @@ function equals(o1, o2) { return false; } +var csp = function() { + if (isDefined(csp.isActive_)) return csp.isActive_; + + var active = !!(document.querySelector('[ng-csp]') || + document.querySelector('[data-ng-csp]')); + + if (!active) { + try { + new Function(''); + } catch (e) { + active = true; + } + } + + return csp.isActive_ = active; +}; -function csp() { - return (document.securityPolicy && document.securityPolicy.isActive) || - (document.querySelector && - !!(document.querySelector('[ng-csp]') || document.querySelector('[data-ng-csp]'))); -} function concat(array1, array2, index) { diff --git a/src/ng/directive/ngCsp.js b/src/ng/directive/ngCsp.js index 38508bb41201..691da10d9a87 100644 --- a/src/ng/directive/ngCsp.js +++ b/src/ng/directive/ngCsp.js @@ -11,8 +11,10 @@ * This is necessary when developing things like Google Chrome Extensions. * * CSP forbids apps to use `eval` or `Function(string)` generated functions (among other things). - * For us to be compatible, we just need to implement the "getterFn" in $parse without violating - * any of these restrictions. + * For Angular to be CSP compatible there are only two things that we need to do differently: + * + * - don't use `Function` constructor to generate optimized value getters + * - don't inject custom stylesheet into the document * * AngularJS uses `Function(string)` generated functions as a speed optimization. Applying the `ngCsp` * directive will cause Angular to use CSP compatibility mode. When this mode is on AngularJS will @@ -23,7 +25,18 @@ * includes some CSS rules (e.g. {@link ng.directive:ngCloak ngCloak}). * To make those directives work in CSP mode, include the `angular-csp.css` manually. * - * In order to use this feature put the `ngCsp` directive on the root element of the application. + * Angular tries to autodetect if CSP is active and automatically turn on the CSP-safe mode. This + * autodetection however triggers a CSP error to be logged in the console: + * + * ``` + * Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of + * script in the following Content Security Policy directive: "default-src 'self'". Note that + * 'script-src' was not explicitly set, so 'default-src' is used as a fallback. + * ``` + * + * This error is harmless but annoying. To prevent the error from showing up, put the `ngCsp` + * directive on the root element of the application or on the `angular.js` script tag, whichever + * appears first in the html document. * * *Note: This directive is only available in the `ng-csp` and `data-ng-csp` attribute form.* * @@ -38,6 +51,6 @@ ``` */ -// ngCsp is not implemented as a proper directive any more, because we need it be processed while we bootstrap -// the system (before $parse is instantiated), for this reason we just have a csp() fn that looks for ng-csp attribute -// anywhere in the current doc +// ngCsp is not implemented as a proper directive any more, because we need it be processed while we +// bootstrap the system (before $parse is instantiated), for this reason we just have +// the csp.isActive() fn that looks for ng-csp attribute anywhere in the current doc diff --git a/test/AngularSpec.js b/test/AngularSpec.js index 6a74c6c615b9..a1d76e772856 100644 --- a/test/AngularSpec.js +++ b/test/AngularSpec.js @@ -413,14 +413,15 @@ describe('angular', function() { describe('csp', function() { - var originalSecurityPolicy; + var originalFunction; beforeEach(function() { - originalSecurityPolicy = document.securityPolicy; + originalFunction = window.Function; }); afterEach(function() { - document.securityPolicy = originalSecurityPolicy; + window.Function = originalFunction; + delete csp.isActive_; }); @@ -430,7 +431,7 @@ describe('angular', function() { it('should return true if CSP is autodetected via CSP v1.1 securityPolicy.isActive property', function() { - document.securityPolicy = {isActive: true}; + window.Function = function() { throw new Error('CSP test') }; expect(csp()).toBe(true); });