Revert "fix(http): exclude caching for authenticated HTTP requests (#…

…54746)" (#55033)

This reverts commit 2258ac7.

Closes: #55031

PR Close #55033

packages/common/http/src/transfer_cache.ts

@@ -101,10 +101,7 @@ export function transferCacheInterceptorFn(
     // POST requests are allowed either globally or at request level
     (requestMethod === 'POST' && !globalOptions.includePostRequests && !requestOptions) ||
     (requestMethod !== 'POST' && !ALLOWED_METHODS.includes(requestMethod)) ||
-    // Do not cache request that require authorization
-    req.headers.has('authorization') ||
-    req.headers.has('proxy-authorization') ||
-    requestOptions === false ||
+    requestOptions === false || //
     globalOptions.filter?.(req) === false
   ) {
     return next(req);

packages/common/http/test/transfer_cache_spec.ts

@@ -261,22 +261,6 @@ describe('TransferCache', () => {
       makeRequestAndExpectNone('/test-2?foo=1', 'POST', {transferCache: true});
     });
 
-    it('should not cache request that requires authorization', async () => {
-      makeRequestAndExpectOne('/test-auth', 'foo', {
-        headers: {Authorization: 'Basic YWxhZGRpbjpvcGVuc2VzYW1l'},
-      });
-
-      makeRequestAndExpectOne('/test-auth', 'foo');
-    });
-
-    it('should not cache request that requires proxy authorization', async () => {
-      makeRequestAndExpectOne('/test-auth', 'foo', {
-        headers: {'Proxy-Authorization': 'Basic YWxhZGRpbjpvcGVuc2VzYW1l'},
-      });
-
-      makeRequestAndExpectOne('/test-auth', 'foo');
-    });
-
     it('should cache POST with the differing body in string form', () => {
       makeRequestAndExpectOne('/test-1', null, {method: 'POST', transferCache: true, body: 'foo'});
       makeRequestAndExpectNone('/test-1', 'POST', {transferCache: true, body: 'foo'});