@e-mc/request 0.8.23 → 0.8.24

package/README.md

@@ -9,7 +9,7 @@
 
 ## Interface
 
-- https://www.unpkg.com/@e-mc/types@0.8.23/lib/index.d.ts
+- https://www.unpkg.com/@e-mc/types@0.8.24/lib/index.d.ts
 
 ```typescript
 import type { IModule, ModuleConstructor } from "./index";
@@ -82,9 +82,9 @@ interface RequestConstructor extends ModuleConstructor {
 
 ## References
 
-- https://www.unpkg.com/@e-mc/types@0.8.23/lib/http.d.ts
-- https://www.unpkg.com/@e-mc/types@0.8.23/lib/request.d.ts
-- https://www.unpkg.com/@e-mc/types@0.8.23/lib/settings.d.ts
+- https://www.unpkg.com/@e-mc/types@0.8.24/lib/http.d.ts
+- https://www.unpkg.com/@e-mc/types@0.8.24/lib/request.d.ts
+- https://www.unpkg.com/@e-mc/types@0.8.24/lib/settings.d.ts
 
 ## LICENSE
 

package/http/adapter/index.js

@@ -1,15 +1,15 @@
 "use strict";
-var _a;
 const fs = require("node:fs");
 const http2 = require("node:http2");
 const yaml = require("js-yaml");
 const types_1 = require("@e-mc/types");
 const module_1 = require("@e-mc/module");
 const util_1 = require("@e-mc/request/util");
-const kOutStream = Symbol('outStream');
-const kOptions = Symbol('options');
 let LOG_TIMEFORMAT = 'readable';
 class HttpAdapter {
+    instance;
+    state;
+    uri;
     static isUnsupported(value) {
         return value === 421 || value === 505;
     }
@@ -38,20 +38,26 @@ class HttpAdapter {
                 break;
         }
     }
+    contentLength = 0;
+    retries = 0;
+    redirects = 0;
+    closed = false;
+    aborted = false;
+    timeout = null;
+    dataTime = null;
+    delayTime = undefined;
+    opts;
+    client;
+    resolve;
+    reject;
+    startTime;
+    #outStream = null;
+    #options;
     constructor(instance, state, uri, options) {
         this.instance = instance;
         this.state = state;
         this.uri = uri;
-        this.contentLength = 0;
-        this.retries = 0;
-        this.redirects = 0;
-        this.closed = false;
-        this.aborted = false;
-        this.timeout = null;
-        this.dataTime = null;
-        this.delayTime = undefined;
-        this[_a] = null;
-        this[kOptions] = options;
+        this.#options = options;
         this.startTime = state.log ? process.hrtime.bigint() : BigInt(0);
         this.setOpts();
     }
@@ -91,14 +97,14 @@ class HttpAdapter {
                     this.retryResponse(statusCode, headers['retry-after']);
                 }
                 else if (HttpAdapter.isUnsupported(statusCode)) {
-                    this.retryDownload(true, this.formatNgFlags(http2.constants.NGHTTP2_PROTOCOL_ERROR, statusCode));
+                    this.retryDownload(true, this.#formatNgFlags(http2.constants.NGHTTP2_PROTOCOL_ERROR, statusCode));
                 }
                 else {
                     switch (flags) {
                         case http2.constants.NGHTTP2_PROTOCOL_ERROR:
                         case http2.constants.NGHTTP2_INADEQUATE_SECURITY:
                         case http2.constants.NGHTTP2_HTTP_1_1_REQUIRED:
-                            this.retryDownload(true, this.formatNgFlags(flags, statusCode, headers.location));
+                            this.retryDownload(true, this.#formatNgFlags(flags, statusCode, headers.location));
                             break;
                         default:
                             this.retryDownload(false, this.formatStatus(statusCode));
@@ -186,7 +192,7 @@ class HttpAdapter {
         if (uri) {
             this.uri = uri;
         }
-        this.opts = this.instance.opts(this.uri, this[kOptions]);
+        this.opts = this.instance.opts(this.uri, this.#options);
     }
     setWriteStream() {
         const pipeTo = this.pipeTo;
@@ -454,7 +460,7 @@ class HttpAdapter {
                 return;
             }
         }
-        this.sendWarning(this.formatRetry((0, util_1.fromStatusCode)(statusCode)));
+        this.sendWarning(this.#formatRetry((0, util_1.fromStatusCode)(statusCode)));
         if ((0, util_1.isRetryable)(statusCode, true)) {
             setImmediate(this.init.bind(this));
         }
@@ -468,7 +474,7 @@ class HttpAdapter {
         return (0, util_1.isRetryable)(value) && ++this.retries <= this.retryLimit;
     }
     retryTimeout() {
-        this.sendWarning(this.formatRetry("HTTP connection timeout"));
+        this.sendWarning(this.#formatRetry("HTTP connection timeout"));
         this.init();
     }
     terminate(err) {
@@ -501,10 +507,10 @@ class HttpAdapter {
             this.outStream = null;
         }
     }
-    formatNgFlags(value, statusCode, location) {
+    #formatNgFlags(value, statusCode, location) {
         return location ? `Using HTTP 1.1 for URL redirect (${location})` : this.formatStatus(statusCode, value ? 'NGHTTP2 Error ' + value : '');
     }
-    formatRetry(message) {
+    #formatRetry(message) {
         return `${message} (${this.retries} / ${this.retryLimit})`;
     }
     set abortController(value) {
@@ -514,13 +520,13 @@ class HttpAdapter {
         return this.opts.outAbort || null;
     }
     set outStream(value) {
-        this[kOutStream] = value;
+        this.#outStream = value;
         if (value) {
             this.opts.outStream = value;
         }
     }
     get outStream() {
-        return this[kOutStream];
+        return this.#outStream;
     }
     get destroyed() {
         return this.client.destroyed || this.httpVersion === 2 && this.client.aborted;
@@ -550,5 +556,4 @@ class HttpAdapter {
         return this.state.config.redirectLimit;
     }
 }
-_a = kOutStream;
 module.exports = HttpAdapter;

package/index.js

@@ -1752,7 +1752,7 @@ class Request extends module_1.default {
         else {
             options = {};
         }
-        const headers = options.headers || (options.headers = {});
+        const headers = (0, util_1.parseOutgoingHeaders)(options.headers) || {};
         for (const attr in headers) {
             const name = attr.toLowerCase();
             if (name === 'content-type' || name === 'content-length') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@e-mc/request",
-  "version": "0.8.23",
+  "version": "0.8.24",
   "description": "Request constructor for E-mc.",
   "main": "index.js",
   "types": "index.d.ts",
@@ -20,8 +20,8 @@
   "license": "MIT",
   "homepage": "https://github.com/anpham6/e-mc#readme",
   "dependencies": {
-    "@e-mc/module": "0.8.23",
-    "@e-mc/types": "0.8.23",
+    "@e-mc/module": "0.8.24",
+    "@e-mc/types": "0.8.24",
     "combined-stream": "^1.0.8",
     "js-yaml": "^4.1.0",
     "picomatch": "^3.0.1",

package/util.d.ts

@@ -4,6 +4,7 @@ import type { IncomingHttpHeaders, OutgoingHttpHeaders } from 'http';
 
 declare namespace util {
     function parseHeader<T = unknown>(headers: IncomingHttpHeaders, name: string): T | undefined;
+    function parseOutgoingHeaders(headers: OutgoingHttpHeaders | Headers | undefined): OutgoingHttpHeaders | undefined;
     function normalizeHeaders(headers: OutgoingHttpHeaders): OutgoingHttpHeaders;
     function getBasicAuth(auth: AuthValue): string;
     function getBasicAuth(username: unknown, password?: unknown): string;

package/util.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.fromSeconds = exports.asFloat = exports.asInt = exports.trimPath = exports.isRetryable = exports.checkRetryable = exports.hasBasicAuth = exports.getBasicAuth = exports.normalizeHeaders = exports.parseHeader = void 0;
+exports.fromSeconds = exports.asFloat = exports.asInt = exports.trimPath = exports.isRetryable = exports.checkRetryable = exports.hasBasicAuth = exports.getBasicAuth = exports.normalizeHeaders = exports.parseOutgoingHeaders = exports.parseHeader = void 0;
 const util = require("util");
 const module_1 = require("@e-mc/module");
 const types_1 = require("@e-mc/types");
@@ -25,6 +25,24 @@ function parseHeader(headers, name) {
     }
 }
 exports.parseHeader = parseHeader;
+function parseOutgoingHeaders(headers) {
+    if (headers) {
+        if (globalThis.Headers && headers instanceof Headers) {
+            const result = {};
+            headers.forEach((value, key) => {
+                if (key === 'set-cookie') {
+                    (result[key] || (result[key] = [])).push(value);
+                }
+                else {
+                    result[key] = value;
+                }
+            });
+            return result;
+        }
+        return headers;
+    }
+}
+exports.parseOutgoingHeaders = parseOutgoingHeaders;
 function normalizeHeaders(headers) {
     const result = Object.create(null);
     for (const name in headers) {
@@ -37,11 +55,11 @@ function normalizeHeaders(headers) {
                 value = value.trim();
                 break;
             default:
-                if (Array.isArray(value)) {
-                    value = value.map(out => module_1.default.asString(out).trim());
-                    break;
+                if (!(0, types_1.isArray)(value)) {
+                    continue;
                 }
-                continue;
+                value = value.map(out => module_1.default.asString(out).trim());
+                break;
         }
         if (value) {
             result[trimPath(name.trim().toLowerCase())] = value;