From e19b04c9ec985821edf1269c628cfa261f81d631 Mon Sep 17 00:00:00 2001 From: Will Moore Date: Fri, 18 Jan 2013 14:09:19 -0800 Subject: 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 --- test/ng/httpBackendSpec.js | 3 +++ 1 file changed, 3 insertions(+) (limited to 'test') diff --git a/test/ng/httpBackendSpec.js b/test/ng/httpBackendSpec.js index a491ae26..a7935a7c 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) { -- cgit v1.2.3