upgrade node-fetch v3 and remove uuid

Author: sofisl

src/TeenyStatistics.ts

@@ -153,7 +153,7 @@ export class TeenyStatistics {
           this._options.concurrentRequests +
           '. Use the TEENY_REQUEST_WARN_CONCURRENT_REQUESTS environment ' +
           'variable or the concurrentRequests option of teeny-request to ' +
-          'increase or disable (0) this warning.',
+          'increase or disable (0) this warning.'
       );
       warning.type = TeenyStatisticsWarning.CONCURRENT_REQUESTS;
       warning.value = this._concurrentRequests;
@@ -187,7 +187,7 @@ export class TeenyStatistics {
     let concurrentRequests = this.DEFAULT_WARN_CONCURRENT_REQUESTS;
 
     const envConcurrentRequests = Number(
-      process.env.TEENY_REQUEST_WARN_CONCURRENT_REQUESTS,
+      process.env.TEENY_REQUEST_WARN_CONCURRENT_REQUESTS
     );
     if (diConcurrentRequests !== undefined) {
       concurrentRequests = diConcurrentRequests;

src/agents.ts

@@ -66,7 +66,7 @@ function shouldUseProxyForURI(uri: string): boolean {
  */
 export function getAgent(
   uri: string,
-  reqOpts: Options,
+  reqOpts: Options
 ): HttpAnyAgent | undefined {
   const isHttp = uri.startsWith('http://');
   const proxy =

src/index.ts

@@ -19,7 +19,6 @@ import {Agent, AgentOptions as HttpsAgentOptions} from 'https';
 import {AgentOptions as HttpAgentOptions} from 'http';
 import type * as f from 'node-fetch' with {'resolution-mode': 'import'};
 import {PassThrough, Readable, pipeline} from 'stream';
-import * as uuid from 'uuid';
 import {getAgent} from './agents';
 import {TeenyStatistics} from './TeenyStatistics';
 // eslint-disable-next-line @typescript-eslint/no-var-requires
@@ -213,7 +212,7 @@ function teenyRequest(
       // TODO: add support for multipart uploads through streaming
       throw new Error('Multipart without callback is not implemented.');
     }
-    const boundary: string = uuid.v4();
+    const boundary: string = globalThis.crypto?.randomUUID();
     (options.headers as Headers)['Content-Type'] =
       `multipart/related; boundary=${boundary}`;
     options.body = createMultipartStream(boundary, multipart);

test/TeenyStatistics.ts

@@ -65,7 +65,7 @@ describe('TeenyStatistics', () => {
     it('should have default concurrent requests', () => {
       assert.strictEqual(
         TeenyStatistics.DEFAULT_WARN_CONCURRENT_REQUESTS,
-        5000,
+        5000
       );
     });
 
@@ -231,8 +231,8 @@ describe('TeenyStatistics', () => {
       t.requestStarting();
       assert(
         emitWarnStub.calledOnceWith(
-          sinon.match.instanceOf(TeenyStatisticsWarning),
-        ),
+          sinon.match.instanceOf(TeenyStatisticsWarning)
+        )
       );
     });
 
@@ -246,8 +246,8 @@ describe('TeenyStatistics', () => {
       t.requestStarting();
       assert(
         emitWarnStub.calledOnceWith(
-          sinon.match.instanceOf(TeenyStatisticsWarning),
-        ),
+          sinon.match.instanceOf(TeenyStatisticsWarning)
+        )
       );
 
       // shouldn't emit on the next call (i.e. still greater than threshold)
@@ -271,8 +271,8 @@ describe('TeenyStatistics', () => {
       t.requestStarting();
       assert(
         emitWarnStub.calledOnceWith(
-          sinon.match.instanceOf(TeenyStatisticsWarning),
-        ),
+          sinon.match.instanceOf(TeenyStatisticsWarning)
+        )
       );
 
       // let's bring the counter back down
@@ -298,7 +298,7 @@ describe('TeenyStatistics', () => {
       assert.strictEqual(warning.value, 5e3);
       assert.strictEqual(
         warning.type,
-        TeenyStatisticsWarning.CONCURRENT_REQUESTS,
+        TeenyStatisticsWarning.CONCURRENT_REQUESTS
       );
     });
 
@@ -311,15 +311,15 @@ describe('TeenyStatistics', () => {
       const errStr: string = emitWarnStub.firstCall.args[0].toString();
       assert(
         errStr.includes('Possible excessive concurrent requests detected.'),
-        'describes the nature of the warning',
+        'describes the nature of the warning'
       );
       assert(
         errStr.includes('TEENY_REQUEST_WARN_CONCURRENT_REQUESTS'),
-        'mentions env var',
+        'mentions env var'
       );
       assert(
         errStr.includes('concurrentRequests'),
-        'mentions concurrentRequests option',
+        'mentions concurrentRequests option'
       );
       assert(errStr.search(/\b0\b/) !== -1, 'mentions 0');
     });

test/agents.ts

@@ -211,7 +211,7 @@ describe('agents', () => {
                 maxSockets: 1000,
               },
             },
-            defaultOptions,
+            defaultOptions
           );
           const agent = getAgent(uri, options);
           assert.strictEqual(agent!.maxSockets, 1000);
@@ -224,7 +224,7 @@ describe('agents', () => {
                 maxSockets: 1000,
               },
             },
-            defaultOptions,
+            defaultOptions
           );
           const agent = getAgent(uri, options);
           assert.strictEqual(agent, undefined);
@@ -243,7 +243,7 @@ describe('agents', () => {
                 maxSockets: 1000,
               },
             },
-            defaultOptions,
+            defaultOptions
           );
           const agent = getAgent(uri, options);
           assert.strictEqual(agent!.maxSockets, 1000);
@@ -256,7 +256,7 @@ describe('agents', () => {
                 maxSockets: 1000,
               },
             },
-            defaultOptions,
+            defaultOptions
           );
           const agent = getAgent(uri, options);
           assert.strictEqual(agent, undefined);