@algolia/requester-node-http 5.1.0 → 5.2.1

package/dist/src/createHttpRequester.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"createHttpRequester.d.ts","sourceRoot":"","sources":["../../src/createHttpRequester.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,KAAK,MAAM,OAAO,CAAC;AAG1B,OAAO,KAAK,EAAc,SAAS,EAAY,MAAM,wBAAwB,CAAC;AAE9E,MAAM,MAAM,0BAA0B,GAAG,OAAO,CAAC;IAC/C,KAAK,EAAE,IAAI,CAAC,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC;IAChC,SAAS,EAAE,IAAI,CAAC,KAAK,CAAC;IACtB,UAAU,EAAE,KAAK,CAAC,KAAK,CAAC;IACxB;;OAEG;IACH,gBAAgB,EAAE,KAAK,CAAC,cAAc,CAAC;CACxC,CAAC,CAAC;AAOH,wBAAgB,mBAAmB,CAAC,EAClC,KAAK,EAAE,eAAe,EACtB,SAAS,EAAE,aAAa,EACxB,UAAU,EAAE,cAAc,EAC1B,gBAAqB,GACtB,GAAE,0BAA+B,GAAG,SAAS,CA6F7C"}
+{"version":3,"file":"createHttpRequester.d.ts","sourceRoot":"","sources":["../../src/createHttpRequester.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,KAAK,MAAM,OAAO,CAAC;AAG1B,OAAO,KAAK,EAAc,SAAS,EAAY,MAAM,wBAAwB,CAAC;AAE9E,MAAM,MAAM,0BAA0B,GAAG,OAAO,CAAC;IAC/C,KAAK,EAAE,IAAI,CAAC,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC;IAChC,SAAS,EAAE,IAAI,CAAC,KAAK,CAAC;IACtB,UAAU,EAAE,KAAK,CAAC,KAAK,CAAC;IACxB;;OAEG;IACH,gBAAgB,EAAE,KAAK,CAAC,cAAc,CAAC;CACxC,CAAC,CAAC;AAOH,wBAAgB,mBAAmB,CAAC,EAClC,KAAK,EAAE,eAAe,EACtB,SAAS,EAAE,aAAa,EACxB,UAAU,EAAE,cAAc,EAC1B,gBAAqB,GACtB,GAAE,0BAA+B,GAAG,SAAS,CAgF7C"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@algolia/requester-node-http",
-  "version": "5.1.0",
+  "version": "5.2.1",
   "description": "Promise-based request library for node using the native http module.",
   "repository": {
     "type": "git",
@@ -23,16 +23,16 @@
     "test": "jest"
   },
   "dependencies": {
-    "@algolia/client-common": "5.1.0"
+    "@algolia/client-common": "5.2.1"
   },
   "devDependencies": {
-    "@babel/preset-env": "7.25.3",
+    "@babel/preset-env": "7.25.4",
     "@babel/preset-typescript": "7.24.7",
     "@types/jest": "29.5.12",
-    "@types/node": "20.14.14",
+    "@types/node": "22.5.0",
     "jest": "29.7.0",
-    "nock": "13.5.4",
-    "ts-jest": "29.2.4",
+    "nock": "13.5.5",
+    "ts-jest": "29.2.5",
     "typescript": "5.5.4"
   },
   "engines": {

package/src/__tests__/node-http-requester.test.ts

@@ -79,7 +79,7 @@ describe('api', () => {
           'my-extra-header': 'algolia',
         }),
       }),
-      expect.any(Function)
+      expect.any(Function),
     );
   });
 });
@@ -88,10 +88,7 @@ describe('status code handling', () => {
   it('sends requests', async () => {
     const body = getStringifiedBody();
 
-    nock(testQueryBaseUrl, { reqheaders: headers })
-      .post('/foo')
-      .query(testQueryHeader)
-      .reply(200, body);
+    nock(testQueryBaseUrl, { reqheaders: headers }).post('/foo').query(testQueryHeader).reply(200, body);
 
     const response = await requester.send(requestStub);
 
@@ -101,10 +98,7 @@ describe('status code handling', () => {
   it('resolves status 200', async () => {
     const body = getStringifiedBody();
 
-    nock(testQueryBaseUrl, { reqheaders: headers })
-      .post('/foo')
-      .query(testQueryHeader)
-      .reply(200, body);
+    nock(testQueryBaseUrl, { reqheaders: headers }).post('/foo').query(testQueryHeader).reply(200, body);
 
     const response = await requester.send(requestStub);
 
@@ -116,10 +110,7 @@ describe('status code handling', () => {
   it('resolves status 300', async () => {
     const reason = 'Multiple Choices';
 
-    nock(testQueryBaseUrl, { reqheaders: headers })
-      .post('/foo')
-      .query(testQueryHeader)
-      .reply(300, reason);
+    nock(testQueryBaseUrl, { reqheaders: headers }).post('/foo').query(testQueryHeader).reply(300, reason);
 
     const response = await requester.send(requestStub);
 
@@ -133,10 +124,7 @@ describe('status code handling', () => {
       message: 'Invalid Application-Id or API-Key',
     });
 
-    nock(testQueryBaseUrl, { reqheaders: headers })
-      .post('/foo')
-      .query(testQueryHeader)
-      .reply(400, body);
+    nock(testQueryBaseUrl, { reqheaders: headers }).post('/foo').query(testQueryHeader).reply(400, body);
 
     const response = await requester.send(requestStub);
 
@@ -157,10 +145,7 @@ describe('status code handling', () => {
 
     const testStream = Readable.from(generate());
 
-    nock(testQueryBaseUrl, { reqheaders: headers })
-      .post('/foo')
-      .query(testQueryHeader)
-      .reply(200, testStream);
+    nock(testQueryBaseUrl, { reqheaders: headers }).post('/foo').query(testQueryHeader).reply(200, testStream);
 
     const response = await requester.send(requestStub);
 

package/src/createHttpRequester.ts

@@ -34,8 +34,7 @@ export function createHttpRequester({
       // eslint-disable-next-line prefer-const -- linter thinks this is not reassigned
       let connectTimeout: NodeJS.Timeout | undefined;
       const url = new URL(request.url);
-      const path =
-        url.search === null ? url.pathname : `${url.pathname}${url.search}`;
+      const path = url.search === null ? url.pathname : `${url.pathname}${url.search}`;
       const options: https.RequestOptions = {
         agent: url.protocol === 'https:' ? httpsAgent : httpAgent,
         hostname: url.hostname,
@@ -52,32 +51,26 @@ export function createHttpRequester({
         options.port = url.port;
       }
 
-      const req = (url.protocol === 'https:' ? https : http).request(
-        options,
-        (response) => {
-          let contentBuffers: Buffer[] = [];
+      const req = (url.protocol === 'https:' ? https : http).request(options, (response) => {
+        let contentBuffers: Buffer[] = [];
 
-          response.on('data', (chunk) => {
-            contentBuffers = contentBuffers.concat(chunk);
-          });
+        response.on('data', (chunk) => {
+          contentBuffers = contentBuffers.concat(chunk);
+        });
 
-          response.on('end', () => {
-            clearTimeout(connectTimeout as NodeJS.Timeout);
-            clearTimeout(responseTimeout as NodeJS.Timeout);
+        response.on('end', () => {
+          clearTimeout(connectTimeout as NodeJS.Timeout);
+          clearTimeout(responseTimeout as NodeJS.Timeout);
 
-            resolve({
-              status: response.statusCode || 0,
-              content: Buffer.concat(contentBuffers).toString(),
-              isTimedOut: false,
-            });
+          resolve({
+            status: response.statusCode || 0,
+            content: Buffer.concat(contentBuffers).toString(),
+            isTimedOut: false,
           });
-        }
-      );
+        });
+      });
 
-      const createTimeout = (
-        timeout: number,
-        content: string
-      ): NodeJS.Timeout => {
+      const createTimeout = (timeout: number, content: string): NodeJS.Timeout => {
         return setTimeout(() => {
           req.destroy();
 
@@ -89,10 +82,7 @@ export function createHttpRequester({
         }, timeout);
       };
 
-      connectTimeout = createTimeout(
-        request.connectTimeout,
-        'Connection timeout'
-      );
+      connectTimeout = createTimeout(request.connectTimeout, 'Connection timeout');
 
       req.on('error', (error) => {
         clearTimeout(connectTimeout as NodeJS.Timeout);
@@ -102,10 +92,7 @@ export function createHttpRequester({
 
       req.once('response', () => {
         clearTimeout(connectTimeout as NodeJS.Timeout);
-        responseTimeout = createTimeout(
-          request.responseTimeout,
-          'Socket timeout'
-        );
+        responseTimeout = createTimeout(request.responseTimeout, 'Socket timeout');
       });
 
       if (request.data !== undefined) {