@faasjs/request 0.0.3-beta.8 → 0.0.3-beta.81

package/README.md

@@ -52,10 +52,10 @@ ___
 | Name | Type | Description |
 | :------ | :------ | :------ |
 | `agent?` | `boolean` | - |
-| `auth?` | `string` | The authentication credentials to use for the request.  Format: `username:password` |
+| `auth?` | `string` | The authentication credentials to use for the request. Format: `username:password` |
 | `body?` | { `[key: string]`: `any`;  } \| `string` | - |
-| `downloadStream?` | `NodeJS.WritableStream` | Create a write stream to download a file. |
-| `file?` | `string` | Path of uploading a file to the server. |
+| `downloadStream?` | `NodeJS.WritableStream` | Create a write stream to download a file. ```ts const stream = createWriteStream('filepath') await request('https://example.com', { downloadStream: stream }) ``` |
+| `file?` | `string` | Path of uploading a file to the server. ```ts await request('https://example.com', { file: 'filepath' }) ``` |
 | `headers?` | `http.OutgoingHttpHeaders` | - |
 | `logger?` | `Logger` | - |
 | `method?` | `string` | The HTTP method to use when making the request. Defaults to GET. |

package/dist/index.d.ts

@@ -39,10 +39,19 @@ type RequestOptions = {
     auth?: string;
     /**
      * Path of uploading a file to the server.
+     *
+     * ```ts
+     * await request('https://example.com', { file: 'filepath' })
+     * ```
      */
     file?: string;
     /**
      * Create a write stream to download a file.
+     *
+     * ```ts
+     * const stream = createWriteStream('filepath')
+     * await request('https://example.com', { downloadStream: stream })
+     * ```
      */
     downloadStream?: NodeJS.WritableStream;
     pfx?: Buffer;

package/dist/index.js

@@ -77,13 +77,20 @@ async function request(url, {
 } = { headers: {} }) {
   if (!logger)
     logger = new import_logger.Logger("request");
-  if (mock)
+  if (mock) {
+    logger.debug("mock %s %j", url, {
+      headers,
+      method,
+      query,
+      body
+    });
     return mock(url, {
       headers,
       method,
       query,
       body
     });
+  }
   if (query) {
     if (!url.includes("?"))
       url += "?";
@@ -136,7 +143,7 @@ async function request(url, {
         res.on("end", () => {
           const data = Buffer.concat(raw).toString();
           logger.timeEnd(url, "response %s %s %s", res.statusCode, res.headers["content-type"], data);
-          const response = /* @__PURE__ */ Object.create(null);
+          const response = res.statusCode >= 200 && res.statusCode < 400 ? /* @__PURE__ */ Object.create(null) : new Error();
           response.request = options;
           response.request.body = body;
           response.statusCode = res.statusCode;
@@ -145,7 +152,7 @@ async function request(url, {
           response.body = data;
           if (response.body && response.headers["content-type"] && response.headers["content-type"].includes("application/json"))
             try {
-              response.body = parse ? parse(response.body) : JSON.parse(response.body);
+              response.body = (parse || JSON.parse)(response.body);
               logger.debug("response.parse JSON");
             } catch (error) {
               console.warn("response plain body", response.body);
@@ -155,6 +162,7 @@ async function request(url, {
             resolve(response);
           else {
             logger.debug("response.error %j", response);
+            response.message = `${res.statusCode} ${res.statusMessage} ${url}`;
             reject(response);
           }
         });

package/dist/index.mjs

@@ -45,13 +45,20 @@ async function request(url, {
 } = { headers: {} }) {
   if (!logger)
     logger = new Logger("request");
-  if (mock)
+  if (mock) {
+    logger.debug("mock %s %j", url, {
+      headers,
+      method,
+      query,
+      body
+    });
     return mock(url, {
       headers,
       method,
       query,
       body
     });
+  }
   if (query) {
     if (!url.includes("?"))
       url += "?";
@@ -104,7 +111,7 @@ async function request(url, {
         res.on("end", () => {
           const data = Buffer.concat(raw).toString();
           logger.timeEnd(url, "response %s %s %s", res.statusCode, res.headers["content-type"], data);
-          const response = /* @__PURE__ */ Object.create(null);
+          const response = res.statusCode >= 200 && res.statusCode < 400 ? /* @__PURE__ */ Object.create(null) : new Error();
           response.request = options;
           response.request.body = body;
           response.statusCode = res.statusCode;
@@ -113,7 +120,7 @@ async function request(url, {
           response.body = data;
           if (response.body && response.headers["content-type"] && response.headers["content-type"].includes("application/json"))
             try {
-              response.body = parse ? parse(response.body) : JSON.parse(response.body);
+              response.body = (parse || JSON.parse)(response.body);
               logger.debug("response.parse JSON");
             } catch (error) {
               console.warn("response plain body", response.body);
@@ -123,6 +130,7 @@ async function request(url, {
             resolve(response);
           else {
             logger.debug("response.error %j", response);
+            response.message = `${res.statusCode} ${res.statusMessage} ${url}`;
             reject(response);
           }
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@faasjs/request",
-  "version": "0.0.3-beta.8",
+  "version": "0.0.3-beta.81",
   "license": "MIT",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -15,14 +15,13 @@
   },
   "funding": "https://github.com/sponsors/faasjs",
   "scripts": {
-    "build": "tsup-node src/index.ts --format esm,cjs",
-    "build:types": "tsup-node src/index.ts --dts-only"
+    "build": "tsup-node src/index.ts --format esm,cjs --dts --clean"
   },
   "files": [
     "dist"
   ],
   "dependencies": {
-    "@faasjs/logger": "^0.0.3-beta.8"
+    "@faasjs/logger": "^0.0.3-beta.81"
   },
   "engines": {
     "npm": ">=8.0.0",