Added hook to address 400 error returned during redirect (#329)

* Added redirect+response hook

During tarball resolution, some redirects would return 400 error if authentication headers are present.  These hooks will intercept a 400 error and remove the authorization header.

* Removed redir caching.  Force delete authentication header on redirect

* Decreases the amount of ts-ignore

Author: Js-Brecht

packages/berry-core/sources/httpUtils.ts

@@ -1,5 +1,5 @@
 import HttpAgent, {HttpsAgent} from 'agentkeepalive';
-import got                     from 'got';
+import got, {GotOptions}       from 'got';
 import tunnel, {ProxyOptions}  from 'tunnel';
 import {URL}                   from 'url';
 
@@ -59,9 +59,25 @@ async function request(target: string, body: Body, {configuration, headers, json
       ? tunnel.httpsOverHttp(parseProxy(httpsProxy))
       : globalHttpsAgent;
 
-  // @ts-ignore
-  const res = await got(target, {agent, body, headers, json, method, encoding: null});
-
+  const gotOptions = {agent, body, headers, json, method, encoding: null};
+
+  const makeHooks = <T extends string | null>() => ({
+    beforeRedirect: [
+      (options: GotOptions<T>) => {
+        if (options.headers && options.headers.authorization) {
+          delete options.headers.authorization;
+        }
+      },
+    ],
+  });
+
+  //@ts-ignore
+  const gotClient = got.extend({
+    ...gotOptions,
+    hooks: makeHooks(),
+  });
+
+  const res = await gotClient(target);
   return await res.body;
 }