fix($http): anonnymous response interceptors should be treated as factories

2 files changed, 12 insertions, 6 deletions

diff --git a/src/service/http.js b/src/service/http.js
index f9a8d921..68f06de7 100644
--- a/src/service/http.js
+++ b/src/service/http.js
@@ -125,7 +125,11 @@ function $HttpProvider() {
         responseInterceptors = [];
 
     forEach(providerResponseInterceptors, function(interceptor) {
-      responseInterceptors.push(isString(interceptor) ? $injector.get(interceptor) : interceptor);
+      responseInterceptors.push(
+          isString(interceptor)
+              ? $injector.get(interceptor)
+              : $injector.invoke(interceptor)
+      );
     });
 
 
diff --git a/test/service/httpSpec.js b/test/service/httpSpec.js
index d827e941..766238d8 100644
--- a/test/service/httpSpec.js
+++ b/test/service/httpSpec.js
@@ -51,11 +51,13 @@ describe('$http', function() {
             };
           });
           // just change the response data and pass the response object along
-          $httpProvider.responseInterceptors.push(function(httpPromise) {
-            return httpPromise.then(function(response) {
-              response.data += '!';
-              return response;
-            });
+          $httpProvider.responseInterceptors.push(function() {
+            return function(httpPromise) {
+              return httpPromise.then(function(response) {
+                response.data += '!';
+                return response;
+              });
+            }
           });
 
           // return a new resolved promise representing modified response object