@wiajs/request 3.0.16 → 3.0.18

package/dist/request.cjs

@@ -1,5 +1,5 @@
 /*!
-  * wia request v3.0.16
+  * wia request v3.0.18
   * (c) 2022-2024 Sibyl Yu and contributors
   * Released under the MIT License.
   */

package/dist/request.mjs

@@ -1,5 +1,5 @@
 /*!
-  * wia request v3.0.16
+  * wia request v3.0.18
   * (c) 2022-2024 Sibyl Yu and contributors
   * Released under the MIT License.
   */

package/lib/index.js

@@ -71,11 +71,8 @@ const log = Log({
  */ function request(uri, options, callback) {
     let R = null;
     try {
-        log({
-            uri,
-            options
-        }, 'request');
         const { opts, cb } = init(uri, options, callback);
+        // log({uri, options, opts}, 'request')
         R = new Request(opts, cb);
     } catch (e) {
         log.err(e, 'request');

package/lib/request.js

@@ -30,10 +30,39 @@ const brotliOptions = {
     finishFlush: zlib.constants.BROTLI_OPERATION_FLUSH
 };
 const isBrotliSupported = utils.isFunction(zlib.createBrotliDecompress);
+// clientRequest 属性转发
+const writeProps = [
+    'protocol',
+    'method',
+    'path',
+    'host',
+    'reusedSocket',
+    'socket',
+    'closed',
+    'destroyed',
+    'writable',
+    'writableAborted',
+    'writableEnded',
+    'writableCorked',
+    'errored',
+    'writableFinished',
+    'writableHighWaterMark',
+    'writableLength',
+    'writableNeedDrain',
+    'writableObjectMode'
+];
+// clientReq 方法转发
+const writeMethods = [
+    'cork',
+    'flushHeaders',
+    'setNoDelay',
+    'setSocketKeepAlive'
+];
 // Create handlers that pass events from native requests
+// 在 clientRequest 事件转发
 const writeEvents = [
-    'abort',
-    'aborted',
+    // 'abort', // 弃用
+    // 'aborted', // 弃用
     'close',
     'connect',
     'continue',
@@ -58,7 +87,7 @@ for (const ev of writeEvents)writeEventEmit[ev] = function(...args) {
     m.redirectReq.emit(ev, ...args) // req 事情映射到 redirectReq 上触发
     ;
 };
-// stream.Readable
+// stream.Readable，在响应流上转发读流取事件
 // data 单独处理
 const readEvents = [
     'close',
@@ -96,7 +125,7 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
         , /** @type {stream.Writable[]} */ this.pipedests = [] // pipe dest
         , /** @type {*} */ this.startTimer = null;
         const m = this;
-        // log({options}, 'constructor');
+        // log({opts}, 'constructor');
         // Initialize the request
         m.sanitizeOptions(opts);
         m.opt = opts;
@@ -121,8 +150,34 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
                 }));
             }
         };
-        // 流模式
+        // Proxy all other public ClientRequest methods 'getHeader'
+        for (const method of writeMethods){
+            // @ts-ignore
+            m[method] = (a, b)=>{
+                log(method, {
+                    a,
+                    b
+                });
+                // @ts-ignore
+                m._currentRequest?.[method](a, b);
+            };
+        }
+        // Proxy all public ClientRequest properties
+        // 'aborted', 'connection' 弃用
+        for (const property of writeProps){
+            Object.defineProperty(m, property, {
+                get () {
+                    // @ts-ignore
+                    const val = m._currentRequest?.[property];
+                    log('get property', {
+                        property
+                    });
+                    return val;
+                }
+            });
+        }
         if (opts.stream) {
+            // 流模式
             // 被 pipe 作为目标时触发，拷贝 src headers
             m.on('pipe', /** @type {stream.Readable} */ (src)=>{
                 // m.ntick &&
@@ -147,7 +202,7 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
             });
         }
     // Perform the first request
-    // m.request(); // 写入数据时执行，否则 pipe时无法写入header
+    // m.request(); // 写入数据时执行，否则 pipe 时无法写入header
     }
     /**
    * Executes the next native request (initial or redirect)
@@ -192,36 +247,6 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
             const req = httpModule.request(opt, m._onResponse);
             m._currentRequest = req;
             req.redirectReq = m;
-            // Proxy all other public ClientRequest methods
-            for (const method of [
-                'flushHeaders',
-                'setNoDelay',
-                'setSocketKeepAlive'
-            ]){
-                m[method] = (a, b)=>{
-                    log(method, {
-                        a,
-                        b
-                    });
-                    m._currentRequest[method](a, b);
-                };
-            }
-            // Proxy all public ClientRequest properties
-            for (const property of [
-                'aborted',
-                'connection',
-                'socket'
-            ]){
-                Object.defineProperty(m, property, {
-                    get () {
-                        const val = m._currentRequest[property];
-                        log('get property', {
-                            property
-                        });
-                        return val;
-                    }
-                });
-            }
             // 启动 startTimer
             if (m.startTimer) m._currentRequest.once('socket', m.startTimer);
             // 接收req事件，转发 到 redirectReq 发射
@@ -268,8 +293,8 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
    */ destroy(error) {
         const m = this;
         if (!m._ended) m.end();
-        else if (m.response) m.response.destroy();
-        else if (m.responseStream) m.responseStream.destroy();
+        if (m.response) m.response.destroy();
+        if (m.responseStream) m.responseStream.destroy();
         // m.clearTimeout();
         destroyRequest(m._currentRequest, error);
         super.destroy(error);
@@ -284,11 +309,11 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
    * @returns {boolean}
    */ write(chunk, encoding, cb) {
         const m = this;
-        log.debug('write', {
+        log({
             data: chunk,
             encoding,
             callback: cb
-        });
+        }, 'write');
         // Writing is not allowed if end has been called
         if (m._ending) throw new WriteAfterEndError();
         // ! 数据写入时连接，pipe 时可设置 header
@@ -474,6 +499,7 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
    * @returns
    */ processResponse(response) {
         const m = this;
+        const { opt } = m;
         // Store the redirected response
         const { statusCode } = response;
         if (m.opt.trackRedirects) {
@@ -491,20 +517,20 @@ const WriteAfterEndError = utils.createErrorType('ERR_STREAM_WRITE_AFTER_END', '
         // even if the specific status code is not understood.
         // If the response is not a redirect; return it as-is
         const { location } = response.headers;
-        log('processResponse', {
+        log({
             statusCode,
             headers: response.headers
-        });
+        }, 'processResponse');
         if (!location || m.opt.followRedirects === false || statusCode < 300 || statusCode >= 400) {
             // 非重定向，返回给原始回调处理
             response.responseUrl = m._currentUrl;
             response.redirects = m._redirects;
-            m.response = response;
+            if (opt.stream) m.response = response;
             // Be a good stream and emit end when the response is finished.
             // Hack to emit end on close because of a core bug that never fires end
             response.on('close', ()=>{
                 if (!m._respended) {
-                    m.response.emit('end');
+                    response.emit('end');
                 }
             });
             response.once('end', ()=>{

package/package.json

@@ -2,7 +2,7 @@
   "name": "@wiajs/request",
   "description": "Stream HTTP request client.",
   "keywords": ["http", "simple", "util", "utility"],
-  "version": "3.0.16",
+  "version": "3.0.18",
   "type": "module",
   "main": "index.js",
   "exports": {
@@ -45,7 +45,7 @@
     "node": ">= 6"
   },
   "dependencies": {
-    "@wiajs/log": "^4.3.10",
+    "@wiajs/log": "^4.3.13",
     "mime-types": "^2.1.35"
   },
   "devDependencies": {