diff options
| author | Will Moore | 2013-01-18 14:09:19 -0800 |
|---|---|---|
| committer | Vojta Jina | 2013-02-14 16:52:02 -0800 |
| commit | 0fa8e47fb5d2df5aab820691696f19662669eed0 (patch) | |
| tree | aecac888230370706c123b21faf844d7fb96d839 /test/ng | |
| parent | 8043784fd73a4768f4e904fd4121b21cd1fb49b4 (diff) | |
| download | angular.js-0fa8e47fb5d2df5aab820691696f19662669eed0.tar.bz2 | |
fix($httpBackend): patch for Firefox bug w/ CORS and response headers
A workaround for https://bugzilla.mozilla.org/show_bug.cgi?id=608735
In FF getAllResponseHeaders() returns null if the request is the result of CORS.
Tried to format the code so that when a FF patch is released and gains enough
traction it can easily be selected and deleted. Heavily inspired by jQuery's
patch for the same bug. This patch falls short of passing through custom headers
but covers all of the "simple response headers" in the spec at
http://www.w3.org/TR/cors/
This commit should get reverted once Firefox 21 gets out.
Closes #1468
Conflicts:
src/ng/httpBackend.js
Diffstat (limited to 'test/ng')
| -rw-r--r-- | test/ng/httpBackendSpec.js | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/test/ng/httpBackendSpec.js b/test/ng/httpBackendSpec.js index 06b63c3c..563b624c 100644 --- a/test/ng/httpBackendSpec.js +++ b/test/ng/httpBackendSpec.js @@ -116,6 +116,9 @@ describe('$httpBackend', function() { }; this.getAllResponseHeaders = valueFn(''); + // for temporary Firefox CORS workaround + // see https://github.com/angular/angular.js/issues/1468 + this.getResponseHeader = valueFn(''); } callback.andCallFake(function(status, response) { |