@faasjs/request 0.0.4-beta.15 → 0.0.4-beta.17

package/dist/index.d.mts

@@ -1,4 +1,4 @@
-import * as http from 'http';
+import * as http from 'node:http';
 import { Logger } from '@faasjs/logger';
 
 type Request = {
@@ -118,4 +118,4 @@ declare class ResponseError extends Error {
  */
 declare function request<T = any>(url: string, { headers, method, query, body, timeout, auth, file, downloadStream, downloadFile, pfx, passphrase, agent, parse, logger, }?: RequestOptions): Promise<Response<T>>;
 
-export { Request, RequestOptions, Response, ResponseError, querystringify, request, setMock };
+export { type Request, type RequestOptions, type Response, ResponseError, querystringify, request, setMock };

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import * as http from 'http';
+import * as http from 'node:http';
 import { Logger } from '@faasjs/logger';
 
 type Request = {
@@ -118,4 +118,4 @@ declare class ResponseError extends Error {
  */
 declare function request<T = any>(url: string, { headers, method, query, body, timeout, auth, file, downloadStream, downloadFile, pfx, passphrase, agent, parse, logger, }?: RequestOptions): Promise<Response<T>>;
 
-export { Request, RequestOptions, Response, ResponseError, querystringify, request, setMock };
+export { type Request, type RequestOptions, type Response, ResponseError, querystringify, request, setMock };

package/dist/index.js

@@ -6,6 +6,8 @@ var url = require('url');
 var fs = require('fs');
 var path = require('path');
 var logger = require('@faasjs/logger');
+var zlib = require('zlib');
+var crypto = require('crypto');
 
 function _interopNamespace(e) {
   if (e && e.__esModule) return e;
@@ -119,6 +121,9 @@ async function request(url$1, {
     passphrase,
     agent
   };
+  if (!options.headers["Accept-Encoding"] && !downloadFile && !downloadStream) {
+    options.headers["Accept-Encoding"] = "br,gzip";
+  }
   for (const key in headers)
     if (typeof headers[key] !== "undefined" && headers[key] !== null)
       options.headers[key] = headers[key];
@@ -129,61 +134,101 @@ async function request(url$1, {
       body = JSON.stringify(body);
   if (body && !options.headers["Content-Length"])
     options.headers["Content-Length"] = Buffer.byteLength(body);
-  return await new Promise(function(resolve, reject) {
+  const requestId = crypto.randomUUID();
+  return await new Promise((resolve, reject) => {
     logger$1.debug("request %j", {
       ...options,
       body
     });
-    const req = protocol.request(options, function(res) {
+    const req = protocol.request(options, (res) => {
       if (downloadStream) {
-        res.pipe(downloadStream);
-        downloadStream.on("finish", () => resolve(void 0));
-      } else if (downloadFile) {
-        const stream = fs.createWriteStream(downloadFile);
-        res.pipe(stream);
-        stream.on("finish", () => resolve(void 0));
-      } else {
-        const raw = [];
-        res.on("data", (chunk) => {
-          raw.push(chunk);
+        res.pipe(downloadStream, { end: true });
+        downloadStream.on("finish", () => {
+          res.destroy();
+          downloadStream.end();
+          resolve(void 0);
         });
+        return;
+      }
+      if (downloadFile) {
+        logger$1.debug("downloadFile");
+        const stream2 = fs.createWriteStream(downloadFile, { autoClose: true });
+        res.pipe(stream2, { end: true });
         res.on("end", () => {
-          const data = Buffer.concat(raw).toString();
-          logger$1.timeEnd(
-            url$1,
-            "response %s %s %s",
-            res.statusCode,
-            res.headers["content-type"],
-            data
+          logger$1.debug("end");
+          res.destroy();
+        });
+        stream2.on("finish", () => {
+          stream2.end();
+          stream2.close();
+          logger$1.debug("finish");
+        });
+        stream2.on("close", () => {
+          logger$1.debug(
+            "finish",
+            res.closed,
+            res.destroyed,
+            stream2.destroyed,
+            stream2.closed,
+            req.closed,
+            req.destroyed
           );
-          const response = /* @__PURE__ */ Object.create(null);
-          response.request = options;
-          response.request.body = body;
-          response.statusCode = res.statusCode;
-          response.statusMessage = res.statusMessage;
-          response.headers = res.headers;
-          response.body = data;
-          if (response.body && response.headers["content-type"] && response.headers["content-type"].includes("application/json"))
-            try {
-              response.body = (parse || JSON.parse)(response.body);
-              logger$1.debug("response.parse JSON");
-            } catch (error) {
-              console.warn("response plain body", response.body);
-              console.error(error);
-            }
-          if (response.statusCode >= 200 && response.statusCode < 400)
-            resolve(response);
-          else {
-            logger$1.debug("response.error %j", response);
-            reject(
-              new ResponseError(
-                `${res.statusMessage || res.statusCode} ${options.host}${options.path}`,
-                response
-              )
-            );
-          }
+          resolve(void 0);
         });
+        return;
       }
+      let stream = res;
+      switch (res.headers["content-encoding"]) {
+        case "br":
+          stream = res.pipe(zlib.createBrotliDecompress());
+          break;
+        case "gzip":
+          stream = res.pipe(zlib.createGunzip());
+          break;
+      }
+      const raw = [];
+      stream.on("data", (chunk) => raw.push(chunk));
+      stream.on("end", () => {
+        const data = Buffer.concat(raw).toString();
+        logger$1.timeEnd(
+          requestId,
+          "response %s %s %s %j",
+          res.statusCode,
+          res.headers["content-type"],
+          res.headers["content-encoding"],
+          data
+        );
+        const response = /* @__PURE__ */ Object.create(null);
+        response.request = options;
+        response.request.body = body;
+        response.statusCode = res.statusCode;
+        response.statusMessage = res.statusMessage;
+        response.headers = res.headers;
+        response.body = data;
+        if (response.body && response.headers["content-type"] && response.headers["content-type"].includes("application/json"))
+          try {
+            response.body = (parse || JSON.parse)(response.body);
+            logger$1.debug("response.parse JSON");
+          } catch (error) {
+            console.warn("response plain body", response.body);
+            console.error(error);
+          }
+        if (response.statusCode >= 200 && response.statusCode < 400)
+          resolve(response);
+        else {
+          logger$1.debug("response.error %j", response);
+          reject(
+            new ResponseError(
+              `${res.statusMessage || res.statusCode} ${options.host}${options.path}`,
+              response
+            )
+          );
+        }
+      });
+      stream.on("error", (e) => {
+        logger$1.timeEnd(requestId, "response.error %j", e);
+        reject(e);
+      });
     });
     if (body)
       req.write(body);
@@ -205,11 +250,16 @@ async function request(url$1, {
       req.setHeader("Content-Length", multipartBody.length);
       req.write(multipartBody);
     }
-    req.on("error", function(e) {
-      logger$1.timeEnd(url$1, "response.error %j", e);
+    req.on("error", (e) => {
+      logger$1.timeEnd(requestId, "response.error %j", e);
       reject(e);
     });
-    logger$1.time(url$1);
+    req.on("timeout", () => {
+      logger$1.timeEnd(requestId, "response.timeout");
+      req.destroy();
+      reject(Error(`Timeout ${url$1}`));
+    });
+    logger$1.time(requestId);
     req.end();
   });
 }

package/dist/index.mjs

@@ -1,9 +1,11 @@
-import * as http from 'http';
-import * as https from 'https';
-import { URL } from 'url';
-import { createWriteStream, readFileSync } from 'fs';
-import { basename } from 'path';
+import * as http from 'node:http';
+import * as https from 'node:https';
+import { URL } from 'node:url';
+import { createWriteStream, readFileSync } from 'node:fs';
+import { basename } from 'node:path';
 import { Logger } from '@faasjs/logger';
+import { createGunzip, createBrotliDecompress } from 'node:zlib';
+import { randomUUID } from 'node:crypto';
 
 // src/index.ts
 var mock = null;
@@ -96,6 +98,9 @@ async function request(url, {
     passphrase,
     agent
   };
+  if (!options.headers["Accept-Encoding"] && !downloadFile && !downloadStream) {
+    options.headers["Accept-Encoding"] = "br,gzip";
+  }
   for (const key in headers)
     if (typeof headers[key] !== "undefined" && headers[key] !== null)
       options.headers[key] = headers[key];
@@ -106,61 +111,101 @@ async function request(url, {
       body = JSON.stringify(body);
   if (body && !options.headers["Content-Length"])
     options.headers["Content-Length"] = Buffer.byteLength(body);
-  return await new Promise(function(resolve, reject) {
+  const requestId = randomUUID();
+  return await new Promise((resolve, reject) => {
     logger.debug("request %j", {
       ...options,
       body
     });
-    const req = protocol.request(options, function(res) {
+    const req = protocol.request(options, (res) => {
       if (downloadStream) {
-        res.pipe(downloadStream);
-        downloadStream.on("finish", () => resolve(void 0));
-      } else if (downloadFile) {
-        const stream = createWriteStream(downloadFile);
-        res.pipe(stream);
-        stream.on("finish", () => resolve(void 0));
-      } else {
-        const raw = [];
-        res.on("data", (chunk) => {
-          raw.push(chunk);
+        res.pipe(downloadStream, { end: true });
+        downloadStream.on("finish", () => {
+          res.destroy();
+          downloadStream.end();
+          resolve(void 0);
         });
+        return;
+      }
+      if (downloadFile) {
+        logger.debug("downloadFile");
+        const stream2 = createWriteStream(downloadFile, { autoClose: true });
+        res.pipe(stream2, { end: true });
         res.on("end", () => {
-          const data = Buffer.concat(raw).toString();
-          logger.timeEnd(
-            url,
-            "response %s %s %s",
-            res.statusCode,
-            res.headers["content-type"],
-            data
+          logger.debug("end");
+          res.destroy();
+        });
+        stream2.on("finish", () => {
+          stream2.end();
+          stream2.close();
+          logger.debug("finish");
+        });
+        stream2.on("close", () => {
+          logger.debug(
+            "finish",
+            res.closed,
+            res.destroyed,
+            stream2.destroyed,
+            stream2.closed,
+            req.closed,
+            req.destroyed
           );
-          const response = /* @__PURE__ */ Object.create(null);
-          response.request = options;
-          response.request.body = body;
-          response.statusCode = res.statusCode;
-          response.statusMessage = res.statusMessage;
-          response.headers = res.headers;
-          response.body = data;
-          if (response.body && response.headers["content-type"] && response.headers["content-type"].includes("application/json"))
-            try {
-              response.body = (parse || JSON.parse)(response.body);
-              logger.debug("response.parse JSON");
-            } catch (error) {
-              console.warn("response plain body", response.body);
-              console.error(error);
-            }
-          if (response.statusCode >= 200 && response.statusCode < 400)
-            resolve(response);
-          else {
-            logger.debug("response.error %j", response);
-            reject(
-              new ResponseError(
-                `${res.statusMessage || res.statusCode} ${options.host}${options.path}`,
-                response
-              )
-            );
-          }
+          resolve(void 0);
         });
+        return;
       }
+      let stream = res;
+      switch (res.headers["content-encoding"]) {
+        case "br":
+          stream = res.pipe(createBrotliDecompress());
+          break;
+        case "gzip":
+          stream = res.pipe(createGunzip());
+          break;
+      }
+      const raw = [];
+      stream.on("data", (chunk) => raw.push(chunk));
+      stream.on("end", () => {
+        const data = Buffer.concat(raw).toString();
+        logger.timeEnd(
+          requestId,
+          "response %s %s %s %j",
+          res.statusCode,
+          res.headers["content-type"],
+          res.headers["content-encoding"],
+          data
+        );
+        const response = /* @__PURE__ */ Object.create(null);
+        response.request = options;
+        response.request.body = body;
+        response.statusCode = res.statusCode;
+        response.statusMessage = res.statusMessage;
+        response.headers = res.headers;
+        response.body = data;
+        if (response.body && response.headers["content-type"] && response.headers["content-type"].includes("application/json"))
+          try {
+            response.body = (parse || JSON.parse)(response.body);
+            logger.debug("response.parse JSON");
+          } catch (error) {
+            console.warn("response plain body", response.body);
+            console.error(error);
+          }
+        if (response.statusCode >= 200 && response.statusCode < 400)
+          resolve(response);
+        else {
+          logger.debug("response.error %j", response);
+          reject(
+            new ResponseError(
+              `${res.statusMessage || res.statusCode} ${options.host}${options.path}`,
+              response
+            )
+          );
+        }
+      });
+      stream.on("error", (e) => {
+        logger.timeEnd(requestId, "response.error %j", e);
+        reject(e);
+      });
     });
     if (body)
       req.write(body);
@@ -182,11 +227,16 @@ async function request(url, {
       req.setHeader("Content-Length", multipartBody.length);
       req.write(multipartBody);
     }
-    req.on("error", function(e) {
-      logger.timeEnd(url, "response.error %j", e);
+    req.on("error", (e) => {
+      logger.timeEnd(requestId, "response.error %j", e);
       reject(e);
     });
-    logger.time(url);
+    req.on("timeout", () => {
+      logger.timeEnd(requestId, "response.timeout");
+      req.destroy();
+      reject(Error(`Timeout ${url}`));
+    });
+    logger.time(requestId);
     req.end();
   });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@faasjs/request",
-  "version": "0.0.4-beta.15",
+  "version": "0.0.4-beta.17",
   "license": "MIT",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -21,10 +21,10 @@
     "dist"
   ],
   "peerDependencies": {
-    "@faasjs/logger": "0.0.4-beta.15"
+    "@faasjs/logger": "0.0.4-beta.17"
   },
   "devDependencies": {
-    "@faasjs/logger": "0.0.4-beta.15"
+    "@faasjs/logger": "0.0.4-beta.17"
   },
   "engines": {
     "npm": ">=9.0.0",