chore: reduce code duplication for agent configuration

Signed-off-by: Manuelraa <[email protected]>
opensearch-project · Jul 18, 2024 · 5077c28 · 5077c28
1 parent 5b5eda5
commit 5077c28
Showing 1 changed file with 15 additions and 19 deletions.
diff --git a/server/auth/types/openid/openid_auth.ts b/server/auth/types/openid/openid_auth.ts
@@ -174,27 +174,23 @@ export class OpenIdAuthentication extends AuthenticationType {
       };
     }
     this.logger.info(getObjectProperties(this.wreckHttpsOption, 'WreckHttpsOptions'));
+
+    // Use proxy agent to allow usage of e.g. http_proxy environment variable
+    const httpAgent = new ProxyAgent();
+    const httpsAllowUnauthorizedAgent = new ProxyAgent({
+      rejectUnauthorized: false,
+    });
+    let httpsAgent = new ProxyAgent();
     if (Object.keys(this.wreckHttpsOption).length > 0) {
-      return wreck.defaults({
-        agents: {
-          http: new ProxyAgent(),
-          https: new ProxyAgent(this.wreckHttpsOption),
-          httpsAllowUnauthorized: new ProxyAgent({
-            rejectUnauthorized: false,
-          }),
-        },
-      });
-    } else {
-      return wreck.defaults({
-        agents: {
-          http: new ProxyAgent(),
-          https: new ProxyAgent(),
-          httpsAllowUnauthorized: new ProxyAgent({
-            rejectUnauthorized: false,
-          }),
-        },
-      });
+      httpsAgent = new ProxyAgent(this.wreckHttpsOption);
     }
+    return wreck.defaults({
+      agents: {
+        http: httpAgent,
+        https: httpsAgent,
+        httpsAllowUnauthorized: httpsAllowUnauthorizedAgent,
+      },
+    });
   }
 
   getWreckHttpsOptions(): WreckHttpsOptions {