Remove HttpsProxyAgent, allow users to pass http.RequestOptions instead

Author: zyc9012

src/config.ts

@@ -1,26 +1,12 @@
-import process from "node:process";
+import http from "node:http";
 
 export type Config = {
   api_key: string | null;
   timeout: number;
-  http_proxy?: string;
-  https_proxy?: string;
-  no_proxy?: string;
+  requestOptions?: http.RequestOptions;
 };
 
-function getEnvVar(name: string): string | undefined {
-  if (typeof Deno !== "undefined") {
-    return Deno.env.get(name);
-  } else if (typeof process !== "undefined") {
-    return process.env[name];
-  }
-  return undefined;
-}
-
 export const config: Config = {
   api_key: null,
   timeout: 60000,
-  http_proxy: getEnvVar("HTTP_PROXY") || getEnvVar("http_proxy"),
-  https_proxy: getEnvVar("HTTPS_PROXY") || getEnvVar("https_proxy"),
-  no_proxy: getEnvVar("NO_PROXY") || getEnvVar("no_proxy"),
 };

src/utils.ts

@@ -2,7 +2,6 @@ import { version } from "../version.ts";
 import https from "node:https";
 import http from "node:http";
 import qs from "node:querystring";
-import { HttpsProxyAgent } from "npm:https-proxy-agent";
 import { RequestTimeoutError } from "./errors.ts";
 import { config } from "./config.ts";
 import { Buffer } from "node:buffer";
@@ -43,7 +42,7 @@ export function getSource() {
 
 export function buildUrl(
   path: string,
-  parameters: qs.ParsedUrlQueryInput,
+  parameters: qs.ParsedUrlQueryInput
 ): string {
   const clonedParams = { ...parameters };
   for (const k in clonedParams) {
@@ -57,23 +56,13 @@ export function buildUrl(
 export function execute(
   path: string,
   parameters: qs.ParsedUrlQueryInput,
-  timeout: number,
+  timeout: number
 ): Promise<string> {
   const url = buildUrl(path, {
     ...parameters,
     source: getSource(),
   });
 
-  // Check if we should use a proxy
-  const urlObj = new URL(url);
-  const shouldUseProxy = !config.no_proxy?.split(",").some((domain) =>
-    urlObj.hostname.endsWith(domain.trim())
-  );
-
-  const proxyUrl = shouldUseProxy
-    ? (urlObj.protocol === "https:" ? config.https_proxy : config.http_proxy)
-    : undefined;
-
   return new Promise((resolve, reject) => {
     let timer: number;
 
@@ -107,18 +96,14 @@ export function execute(
       if (timer) clearTimeout(timer);
     };
 
-    const options: https.RequestOptions = {
-      timeout: timeout > 0 ? timeout : undefined,
-    };
-
-    if (proxyUrl) {
-      options.agent = new HttpsProxyAgent(proxyUrl);
-    }
+    const options =
+      (parameters.requestOptions as http.RequestOptions) ||
+      config.requestOptions ||
+      {};
 
-    const req = https.get(url, options, handleResponse).on(
-      "error",
-      handleError,
-    );
+    const req = https
+      .get(url, options, handleResponse)
+      .on("error", handleError);
 
     if (timeout > 0) {
       timer = setTimeout(() => {