feat: Support URL objects

src/common.ts

@@ -118,15 +118,15 @@
 }
 
 export interface Headers {
   [index: string]: any;
 }
 export type GaxiosPromise<T = any> = Promise<GaxiosResponse<T>>;
 
 export interface GaxiosXMLHttpRequest {
   responseURL: string;
 }
 
 export interface GaxiosResponse<T = any> {
   config: GaxiosOptions;
   data: T;
   status: number;
@@ -143,13 +143,16 @@
    * Optional method to override making the actual HTTP request. Useful
    * for writing tests.
    */
   adapter?: <T = any>(
     options: GaxiosOptions,
     defaultAdapter: (options: GaxiosOptions) => GaxiosPromise<T>
   ) => GaxiosPromise<T>;
-  url?: string;
-  baseUrl?: string; // deprecated
-  baseURL?: string;
+  url?: string | URL;
+  /**
+   * @deprecated
+   */
+  baseUrl?: string;
+  baseURL?: string | URL;
   method?:
     | 'GET'
     | 'HEAD'
@@ -161,8 +164,8 @@
     | 'TRACE'
     | 'PATCH';
   headers?: Headers;
   data?: any;
   body?: any;
   /**
    * The maximum size of the http response content in bytes allowed.
    */
@@ -172,13 +175,13 @@
    */
   maxRedirects?: number;
   follow?: number;
   params?: any;
   paramsSerializer?: (params: {[index: string]: string | number}) => string;
   timeout?: number;
   /**
    * @deprecated ignored
    */
   onUploadProgress?: (progressEvent: any) => void;
   responseType?:
     | 'arraybuffer'
     | 'blob'
@@ -192,7 +195,7 @@
   retry?: boolean;
   // Should be instance of https://developer.mozilla.org/en-US/docs/Web/API/AbortSignal
   // interface. Left as 'any' due to incompatibility between spec and abort-controller.
   signal?: any;
   size?: number;
   /**
    * Implementation of `fetch` to use when making the API call. By default,
@@ -224,7 +227,7 @@
  *
  * @experimental
  */
 export type RedactableGaxiosResponse<T = any> = Pick<
   GaxiosResponse<T>,
   'config' | 'data' | 'headers'
 >;
@@ -395,7 +398,8 @@
     redactObject(data.config.body);
 
     try {
-      const url = new URL(data.config.url || '');
+      const url = new URL('', data.config.url);
+
       if (url.searchParams.has('token')) {
         url.searchParams.set('token', REDACT);
       }

src/gaxios.ts

@@ -79,13 +79,13 @@ function loadProxy() {
 
 loadProxy();
 
-function skipProxy(url: string) {
+function skipProxy(url: string | URL) {
   const noProxyEnv = process.env.NO_PROXY ?? process.env.no_proxy;
   if (!noProxyEnv) {
     return false;
   }
   const noProxyUrls = noProxyEnv.split(',');
-  const parsedURL = new URL(url);
+  const parsedURL = url instanceof URL ? url : new URL(url);
   return !!noProxyUrls.find(url => {
     if (url.startsWith('*.') || url.startsWith('.')) {
       url = url.replace(/^\*\./, '.');
@@ -98,7 +98,7 @@ function skipProxy(url: string) {
 
 // Figure out if we should be using a proxy. Only if it's required, load
 // the https-proxy-agent module as it adds startup cost.
-function getProxy(url: string) {
+function getProxy(url: string | URL) {
   // If there is a match between the no_proxy env variables and the url, then do not proxy
   if (skipProxy(url)) {
     return undefined;
@@ -239,7 +239,7 @@ export class Gaxios {
     // baseUrl has been deprecated, remove in 2.0
     const baseUrl = opts.baseUrl || opts.baseURL;
     if (baseUrl) {
-      opts.url = baseUrl + opts.url;
+      opts.url = baseUrl.toString() + opts.url;
     }
 
     opts.paramsSerializer = opts.paramsSerializer || this.paramsSerializer;
@@ -248,7 +248,7 @@ export class Gaxios {
       if (additionalQueryParams.startsWith('?')) {
         additionalQueryParams = additionalQueryParams.slice(1);
       }
-      const prefix = opts.url.includes('?') ? '&' : '?';
+      const prefix = opts.url.toString().includes('?') ? '&' : '?';
       opts.url = opts.url + prefix + additionalQueryParams;
     }
 

test/test.getch.ts

@@ -136,6 +136,13 @@ describe('🚙 error handling', () => {
 });
 
 describe('🥁 configuration options', () => {
+  it('should accept URL objects', async () => {
+    const scope = nock(url).get('/').reply(204);
+    const res = await request({url: new URL(url)});
+    scope.done();
+    assert.strictEqual(res.config.method, 'GET');
+  });
+
   it('should use options passed into the constructor', async () => {
     const scope = nock(url).head('/').reply(200);
     const inst = new Gaxios({method: 'HEAD'});