urllib 3.25.0 → 3.26.0

package/README.md

@@ -1,19 +1,19 @@
-# urllib
+# urllib@3
 
 [![NPM version][npm-image]][npm-url]
-[![Node.js CI](https://github.com/node-modules/urllib/actions/workflows/nodejs.yml/badge.svg)](https://github.com/node-modules/urllib/actions/workflows/nodejs.yml)
+[![CI for 3.x](https://github.com/node-modules/urllib/actions/workflows/nodejs-3.x.yml/badge.svg?branch=3.x)](https://github.com/node-modules/urllib/actions/workflows/nodejs-3.x.yml)
 [![Test coverage][codecov-image]][codecov-url]
 [![Known Vulnerabilities][snyk-image]][snyk-url]
 [![npm download][download-image]][download-url]
 
-[npm-image]: https://img.shields.io/npm/v/urllib.svg?style=flat-square
-[npm-url]: https://npmjs.org/package/urllib
-[codecov-image]: https://codecov.io/gh/node-modules/urllib/branch/master/graph/badge.svg
-[codecov-url]: https://codecov.io/gh/node-modules/urllib
+[npm-image]: https://img.shields.io/npm/v/urllib/release-3.x.svg?style=flat-square
+[npm-url]: https://npmjs.org/package/urllib/v/release-3.x
+[codecov-image]: https://codecov.io/gh/node-modules/urllib/branch/3.x/graph/badge.svg
+[codecov-url]: https://app.codecov.io/gh/node-modules/urllib/tree/3.x
 [snyk-image]: https://snyk.io/test/npm/urllib/badge.svg?style=flat-square
 [snyk-url]: https://snyk.io/test/npm/urllib
 [download-image]: https://img.shields.io/npm/dm/urllib.svg?style=flat-square
-[download-url]: https://npmjs.org/package/urllib
+[download-url]: https://npmjs.org/package/urllib/v/release-3.x
 
 Request HTTP URLs in a complex world — basic
 and digest authentication, redirections, timeout and more.
@@ -21,7 +21,7 @@ and digest authentication, redirections, timeout and more.
 ## Install
 
 ```bash
-npm install urllib
+npm install urllib@3
 ```
 
 ## Usage
@@ -209,6 +209,22 @@ Response is normal object, it contains:
 NODE_DEBUG=urllib:* npm test
 ```
 
+## Request with HTTP2
+
+Create a HttpClient with `options.allowH2 = true`
+
+```ts
+import { HttpClient } from 'urllib';
+
+const httpClient = new HttpClient({
+  allowH2: true,
+});
+
+const response = await httpClient.request('https://node.js.org');
+console.log(response.status);
+console.log(response.headers);
+```
+
 ## Mocking Request
 
 export from [undici](https://undici.nodejs.org/#/docs/best-practices/mocking-request)
@@ -283,6 +299,9 @@ Fork [undici benchmarks script](https://github.com/fengmk2/undici/blob/urllib-be
 | undici - stream     |     45 | 12523.45 req/sec | ± 2.97 % |             + 754.61 % |
 | undici - dispatch   |     51 | 12970.18 req/sec | ± 3.15 % |             + 785.10 % |
 
+## License
+
+[MIT](LICENSE)
 
 <!-- GITCONTRIBUTOR_START -->
 
@@ -301,7 +320,3 @@ Fork [undici benchmarks script](https://github.com/fengmk2/undici/blob/urllib-be
 This project follows the git-contributor [spec](https://github.com/xudafeng/git-contributor), auto updated at `Mon Dec 04 2023 00:13:39 GMT+0800`.
 
 <!-- GITCONTRIBUTOR_END -->
-
-## License
-
-[MIT](LICENSE)

package/dist/commonjs/HttpAgent.d.ts

@@ -1,4 +1,3 @@
-/// <reference types="node" />
 import { LookupFunction } from 'node:net';
 import { Agent, Dispatcher, buildConnector } from 'undici';
 export type CheckAddressFunction = (ip: string, family: number | string) => boolean;
@@ -6,6 +5,7 @@ export type HttpAgentOptions = {
     lookup?: LookupFunction;
     checkAddress?: CheckAddressFunction;
     connect?: buildConnector.BuildOptions;
+    allowH2?: boolean;
 };
 export declare class HttpAgent extends Agent {
     #private;

package/dist/commonjs/HttpAgent.js

@@ -54,7 +54,7 @@ class HttpAgent extends undici_1.Agent {
             });
         };
         super({
-            connect: { ...options.connect, lookup },
+            connect: { ...options.connect, lookup, allowH2: options.allowH2 },
         });
         this.#checkAddress = options.checkAddress;
     }

package/dist/commonjs/HttpClient.d.ts

@@ -1,6 +1,3 @@
-/// <reference types="node" />
-/// <reference types="node" />
-/// <reference types="node" />
 import { EventEmitter } from 'node:events';
 import { LookupFunction } from 'node:net';
 import { Dispatcher } from 'undici';
@@ -9,6 +6,8 @@ import { RequestURL, RequestOptions, RequestMeta } from './Request.js';
 import { RawResponseWithMeta, HttpClientResponse } from './Response.js';
 export type ClientOptions = {
     defaultArgs?: RequestOptions;
+    /** Allow to use HTTP2 first. Default is `false` */
+    allowH2?: boolean;
     /**
      * Custom DNS lookup function, default is `dns.lookup`.
      */

package/dist/commonjs/HttpClient.js

@@ -68,7 +68,7 @@ class BlobFromStream {
         return 'Blob';
     }
 }
-exports.HEADER_USER_AGENT = (0, default_user_agent_1.default)('node-urllib', '3.25.0');
+exports.HEADER_USER_AGENT = (0, default_user_agent_1.default)('node-urllib', '3.26.0');
 function getFileName(stream) {
     const filePath = stream.path;
     if (filePath) {
@@ -94,11 +94,19 @@ class HttpClient extends node_events_1.EventEmitter {
                 lookup: clientOptions.lookup,
                 checkAddress: clientOptions.checkAddress,
                 connect: clientOptions.connect,
+                allowH2: clientOptions.allowH2,
             });
         }
         else if (clientOptions?.connect) {
             this.#dispatcher = new undici_1.Agent({
                 connect: clientOptions.connect,
+                allowH2: clientOptions.allowH2,
+            });
+        }
+        else if (clientOptions?.allowH2) {
+            // Support HTTP2
+            this.#dispatcher = new undici_1.Agent({
+                allowH2: clientOptions.allowH2,
             });
         }
         (0, diagnosticsChannel_js_1.initDiagnosticsChannel)();

package/dist/commonjs/IncomingHttpHeaders.d.ts

@@ -1,4 +1,3 @@
-/// <reference types="node" />
 import type { Except } from 'type-fest';
 import type { IncomingHttpHeaders as HTTPIncomingHttpHeaders } from 'node:http';
 export interface IncomingHttpHeaders extends Except<HTTPIncomingHttpHeaders, 'set-cookie'> {

package/dist/commonjs/Request.d.ts

@@ -1,7 +1,3 @@
-/// <reference types="node" />
-/// <reference types="node" />
-/// <reference types="node" />
-/// <reference types="node" />
 import type { Readable, Writable } from 'node:stream';
 import type { EventEmitter } from 'node:events';
 import type { Dispatcher } from 'undici';

package/dist/commonjs/Response.d.ts

@@ -1,4 +1,3 @@
-/// <reference types="node" />
 import type { Readable } from 'node:stream';
 import type { IncomingHttpHeaders } from './IncomingHttpHeaders.js';
 export type SocketInfo = {

package/dist/commonjs/diagnosticsChannel.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.initDiagnosticsChannel = void 0;
+exports.initDiagnosticsChannel = initDiagnosticsChannel;
 const node_diagnostics_channel_1 = __importDefault(require("node:diagnostics_channel"));
 const node_perf_hooks_1 = require("node:perf_hooks");
 const node_util_1 = require("node:util");
@@ -179,8 +179,13 @@ function initDiagnosticsChannel() {
             return;
         // get socket from opaque
         const socket = opaque[symbols_js_1.default.kRequestSocket];
-        socket[symbols_js_1.default.kHandledResponses]++;
-        debug('[%s] Request#%d get %s response headers on Socket#%d (handled %d responses, sock: %o)', name, opaque[symbols_js_1.default.kRequestId], response.statusCode, socket[symbols_js_1.default.kSocketId], socket[symbols_js_1.default.kHandledResponses], formatSocket(socket));
+        if (socket) {
+            socket[symbols_js_1.default.kHandledResponses]++;
+            debug('[%s] Request#%d get %s response headers on Socket#%d (handled %d responses, sock: %o)', name, opaque[symbols_js_1.default.kRequestId], response.statusCode, socket[symbols_js_1.default.kSocketId], socket[symbols_js_1.default.kHandledResponses], formatSocket(socket));
+        }
+        else {
+            debug('[%s] Request#%d get %s response headers on Unknown Socket', name, opaque[symbols_js_1.default.kRequestId], response.statusCode);
+        }
         if (!opaque[symbols_js_1.default.kEnableRequestTiming])
             return;
         opaque[symbols_js_1.default.kRequestTiming].waiting = (0, utils_js_1.performanceTime)(opaque[symbols_js_1.default.kRequestStartTime]);
@@ -189,8 +194,9 @@ function initDiagnosticsChannel() {
     subscribe('undici:request:trailers', (message, name) => {
         const { request } = message;
         const opaque = getRequestOpaque(request, kHandler);
-        if (!opaque || !opaque[symbols_js_1.default.kRequestId])
+        if (!opaque || !opaque[symbols_js_1.default.kRequestId]) {
             return;
+        }
         debug('[%s] Request#%d get response body and trailers', name, opaque[symbols_js_1.default.kRequestId]);
         if (!opaque[symbols_js_1.default.kEnableRequestTiming])
             return;
@@ -207,4 +213,3 @@ function initDiagnosticsChannel() {
     //     formatSocket(socket), error);
     // });
 }
-exports.initDiagnosticsChannel = initDiagnosticsChannel;

package/dist/commonjs/index.js

@@ -17,7 +17,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.USER_AGENT = exports.HttpClient2 = exports.HttpClient = exports.getGlobalDispatcher = exports.setGlobalDispatcher = exports.Dispatcher = exports.Agent = exports.ProxyAgent = exports.MockAgent = exports.curl = exports.request = exports.getDefaultHttpClient = void 0;
+exports.USER_AGENT = exports.HttpClient2 = exports.HttpClient = exports.getGlobalDispatcher = exports.setGlobalDispatcher = exports.Dispatcher = exports.Agent = exports.ProxyAgent = exports.MockAgent = void 0;
+exports.getDefaultHttpClient = getDefaultHttpClient;
+exports.request = request;
+exports.curl = curl;
 const ylru_1 = __importDefault(require("ylru"));
 const HttpClient_js_1 = require("./HttpClient.js");
 let httpClient;
@@ -28,7 +31,6 @@ function getDefaultHttpClient() {
     }
     return httpClient;
 }
-exports.getDefaultHttpClient = getDefaultHttpClient;
 async function request(url, options) {
     if (options?.socketPath) {
         let domainSocketHttpclient = domainSocketHttpClients.get(options.socketPath);
@@ -42,7 +44,6 @@ async function request(url, options) {
     }
     return await getDefaultHttpClient().request(url, options);
 }
-exports.request = request;
 // export curl method is keep compatible with urllib.curl()
 // ```ts
 // import * as urllib from 'urllib';
@@ -51,7 +52,6 @@ exports.request = request;
 async function curl(url, options) {
     return await request(url, options);
 }
-exports.curl = curl;
 var undici_1 = require("undici");
 Object.defineProperty(exports, "MockAgent", { enumerable: true, get: function () { return undici_1.MockAgent; } });
 Object.defineProperty(exports, "ProxyAgent", { enumerable: true, get: function () { return undici_1.ProxyAgent; } });

package/dist/commonjs/utils.js

@@ -1,6 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.isReadable = exports.performanceTime = exports.globalId = exports.digestAuthHeader = exports.sleep = exports.parseJSON = void 0;
+exports.parseJSON = parseJSON;
+exports.sleep = sleep;
+exports.digestAuthHeader = digestAuthHeader;
+exports.globalId = globalId;
+exports.performanceTime = performanceTime;
+exports.isReadable = isReadable;
 const node_crypto_1 = require("node:crypto");
 const node_stream_1 = require("node:stream");
 const node_perf_hooks_1 = require("node:perf_hooks");
@@ -49,13 +54,11 @@ function parseJSON(data, fixJSONCtlChars) {
     }
     return data;
 }
-exports.parseJSON = parseJSON;
 function sleep(ms) {
     return new Promise(resolve => {
         setTimeout(resolve, ms);
     });
 }
-exports.sleep = sleep;
 function md5(s) {
     const sum = (0, node_crypto_1.createHash)('md5');
     sum.update(s, 'utf8');
@@ -123,7 +126,6 @@ function digestAuthHeader(method, uri, wwwAuthenticate, userpass) {
     }
     return authstring;
 }
-exports.digestAuthHeader = digestAuthHeader;
 const MAX_ID_VALUE = Math.pow(2, 31) - 10;
 const globalIds = {};
 function globalId(category) {
@@ -132,11 +134,9 @@ function globalId(category) {
     }
     return ++globalIds[category];
 }
-exports.globalId = globalId;
 function performanceTime(startTime, now) {
     return Math.floor(((now ?? node_perf_hooks_1.performance.now()) - startTime) * 1000) / 1000;
 }
-exports.performanceTime = performanceTime;
 function isReadable(stream) {
     if (typeof node_stream_1.Readable.isReadable === 'function')
         return node_stream_1.Readable.isReadable(stream);
@@ -150,4 +150,3 @@ function isReadable(stream) {
         && typeof stream._read === 'function'
         && typeof stream._readableState === 'object';
 }
-exports.isReadable = isReadable;

package/dist/esm/HttpAgent.d.ts

@@ -1,4 +1,3 @@
-/// <reference types="node" resolution-mode="require"/>
 import { LookupFunction } from 'node:net';
 import { Agent, Dispatcher, buildConnector } from 'undici';
 export type CheckAddressFunction = (ip: string, family: number | string) => boolean;
@@ -6,6 +5,7 @@ export type HttpAgentOptions = {
     lookup?: LookupFunction;
     checkAddress?: CheckAddressFunction;
     connect?: buildConnector.BuildOptions;
+    allowH2?: boolean;
 };
 export declare class HttpAgent extends Agent {
     #private;

package/dist/esm/HttpAgent.js

@@ -48,7 +48,7 @@ export class HttpAgent extends Agent {
             });
         };
         super({
-            connect: { ...options.connect, lookup },
+            connect: { ...options.connect, lookup, allowH2: options.allowH2 },
         });
         this.#checkAddress = options.checkAddress;
     }

package/dist/esm/HttpClient.d.ts

@@ -1,6 +1,3 @@
-/// <reference types="node" resolution-mode="require"/>
-/// <reference types="node" resolution-mode="require"/>
-/// <reference types="node" resolution-mode="require"/>
 import { EventEmitter } from 'node:events';
 import { LookupFunction } from 'node:net';
 import { Dispatcher } from 'undici';
@@ -9,6 +6,8 @@ import { RequestURL, RequestOptions, RequestMeta } from './Request.js';
 import { RawResponseWithMeta, HttpClientResponse } from './Response.js';
 export type ClientOptions = {
     defaultArgs?: RequestOptions;
+    /** Allow to use HTTP2 first. Default is `false` */
+    allowH2?: boolean;
     /**
      * Custom DNS lookup function, default is `dns.lookup`.
      */

package/dist/esm/HttpClient.js

@@ -62,7 +62,7 @@ class BlobFromStream {
         return 'Blob';
     }
 }
-export const HEADER_USER_AGENT = createUserAgent('node-urllib', '3.25.0');
+export const HEADER_USER_AGENT = createUserAgent('node-urllib', '3.26.0');
 function getFileName(stream) {
     const filePath = stream.path;
     if (filePath) {
@@ -88,11 +88,19 @@ export class HttpClient extends EventEmitter {
                 lookup: clientOptions.lookup,
                 checkAddress: clientOptions.checkAddress,
                 connect: clientOptions.connect,
+                allowH2: clientOptions.allowH2,
             });
         }
         else if (clientOptions?.connect) {
             this.#dispatcher = new Agent({
                 connect: clientOptions.connect,
+                allowH2: clientOptions.allowH2,
+            });
+        }
+        else if (clientOptions?.allowH2) {
+            // Support HTTP2
+            this.#dispatcher = new Agent({
+                allowH2: clientOptions.allowH2,
             });
         }
         initDiagnosticsChannel();

package/dist/esm/IncomingHttpHeaders.d.ts

@@ -1,4 +1,3 @@
-/// <reference types="node" resolution-mode="require"/>
 import type { Except } from 'type-fest';
 import type { IncomingHttpHeaders as HTTPIncomingHttpHeaders } from 'node:http';
 export interface IncomingHttpHeaders extends Except<HTTPIncomingHttpHeaders, 'set-cookie'> {

package/dist/esm/Request.d.ts

@@ -1,7 +1,3 @@
-/// <reference types="node" resolution-mode="require"/>
-/// <reference types="node" resolution-mode="require"/>
-/// <reference types="node" resolution-mode="require"/>
-/// <reference types="node" resolution-mode="require"/>
 import type { Readable, Writable } from 'node:stream';
 import type { EventEmitter } from 'node:events';
 import type { Dispatcher } from 'undici';

package/dist/esm/Response.d.ts

@@ -1,4 +1,3 @@
-/// <reference types="node" resolution-mode="require"/>
 import type { Readable } from 'node:stream';
 import type { IncomingHttpHeaders } from './IncomingHttpHeaders.js';
 export type SocketInfo = {

package/dist/esm/diagnosticsChannel.js

@@ -173,8 +173,13 @@ export function initDiagnosticsChannel() {
             return;
         // get socket from opaque
         const socket = opaque[symbols.kRequestSocket];
-        socket[symbols.kHandledResponses]++;
-        debug('[%s] Request#%d get %s response headers on Socket#%d (handled %d responses, sock: %o)', name, opaque[symbols.kRequestId], response.statusCode, socket[symbols.kSocketId], socket[symbols.kHandledResponses], formatSocket(socket));
+        if (socket) {
+            socket[symbols.kHandledResponses]++;
+            debug('[%s] Request#%d get %s response headers on Socket#%d (handled %d responses, sock: %o)', name, opaque[symbols.kRequestId], response.statusCode, socket[symbols.kSocketId], socket[symbols.kHandledResponses], formatSocket(socket));
+        }
+        else {
+            debug('[%s] Request#%d get %s response headers on Unknown Socket', name, opaque[symbols.kRequestId], response.statusCode);
+        }
         if (!opaque[symbols.kEnableRequestTiming])
             return;
         opaque[symbols.kRequestTiming].waiting = performanceTime(opaque[symbols.kRequestStartTime]);
@@ -183,8 +188,9 @@ export function initDiagnosticsChannel() {
     subscribe('undici:request:trailers', (message, name) => {
         const { request } = message;
         const opaque = getRequestOpaque(request, kHandler);
-        if (!opaque || !opaque[symbols.kRequestId])
+        if (!opaque || !opaque[symbols.kRequestId]) {
             return;
+        }
         debug('[%s] Request#%d get response body and trailers', name, opaque[symbols.kRequestId]);
         if (!opaque[symbols.kEnableRequestTiming])
             return;

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "urllib",
-  "version": "3.25.0",
+  "version": "3.26.0",
   "publishConfig": {
-    "tag": "latest"
+    "access": "public"
   },
-  "description": "Help in opening URLs (mostly HTTP) in a complex world — basic and digest authentication, redirections, cookies and more. Base undici fetch API.",
+  "description": "Help in opening URLs (mostly HTTP) in a complex world — basic and digest authentication, redirections, timeout and more. Base undici API.",
   "keywords": [
     "urllib",
     "http",
@@ -40,7 +40,7 @@
     "test-keepalive": "cross-env TEST_KEEPALIVE_COUNT=50 vitest run --test-timeout 180000 keep-alive-header.test.ts",
     "cov": "vitest run --coverage",
     "preci": "node scripts/pre_test.js",
-    "ci": "npm run lint && npm run cov && node scripts/build_test.js",
+    "ci": "npm run lint && npm run cov && node scripts/build_test.js && attw --pack --ignore-rules no-resolution",
     "contributor": "git-contributor",
     "clean": "rm -rf dist",
     "prepublishOnly": "npm run build"
@@ -59,6 +59,7 @@
     "ylru": "^1.3.2"
   },
   "devDependencies": {
+    "@arethetypeswrong/cli": "^0.15.3",
     "@tsconfig/node18": "^18.2.1",
     "@tsconfig/strictest": "^2.0.2",
     "@types/busboy": "^1.5.0",
@@ -98,10 +99,12 @@
   "exports": {
     ".": {
       "import": {
+        "source": "./src/index.ts",
         "types": "./dist/esm/index.d.ts",
         "default": "./dist/esm/index.js"
       },
       "require": {
+        "source": "./src/index.ts",
         "types": "./dist/commonjs/index.d.ts",
         "default": "./dist/commonjs/index.js"
       }

package/src/HttpAgent.ts

@@ -12,6 +12,7 @@ export type HttpAgentOptions = {
   lookup?: LookupFunction;
   checkAddress?: CheckAddressFunction;
   connect?: buildConnector.BuildOptions,
+  allowH2?: boolean;
 };
 
 class IllegalAddressError extends Error {
@@ -62,7 +63,7 @@ export class HttpAgent extends Agent {
       });
     };
     super({
-      connect: { ...options.connect, lookup },
+      connect: { ...options.connect, lookup, allowH2: options.allowH2 },
     });
     this.#checkAddress = options.checkAddress;
   }

package/src/HttpClient.ts

@@ -70,6 +70,8 @@ const isNode14Or16 = /v1[46]\./.test(process.version);
 
 export type ClientOptions = {
   defaultArgs?: RequestOptions;
+  /** Allow to use HTTP2 first. Default is `false` */
+  allowH2?: boolean;
   /**
    * Custom DNS lookup function, default is `dns.lookup`.
    */
@@ -187,10 +189,17 @@ export class HttpClient extends EventEmitter {
         lookup: clientOptions.lookup,
         checkAddress: clientOptions.checkAddress,
         connect: clientOptions.connect,
+        allowH2: clientOptions.allowH2,
       });
     } else if (clientOptions?.connect) {
       this.#dispatcher = new Agent({
         connect: clientOptions.connect,
+        allowH2: clientOptions.allowH2,
+      });
+    } else if (clientOptions?.allowH2) {
+      // Support HTTP2
+      this.#dispatcher = new Agent({
+        allowH2: clientOptions.allowH2,
       });
     }
     initDiagnosticsChannel();

package/src/diagnosticsChannel.ts

@@ -184,10 +184,15 @@ export function initDiagnosticsChannel() {
 
     // get socket from opaque
     const socket = opaque[symbols.kRequestSocket];
-    socket[symbols.kHandledResponses]++;
-    debug('[%s] Request#%d get %s response headers on Socket#%d (handled %d responses, sock: %o)',
-      name, opaque[symbols.kRequestId], response.statusCode, socket[symbols.kSocketId], socket[symbols.kHandledResponses],
-      formatSocket(socket));
+    if (socket) {
+      socket[symbols.kHandledResponses]++;
+      debug('[%s] Request#%d get %s response headers on Socket#%d (handled %d responses, sock: %o)',
+        name, opaque[symbols.kRequestId], response.statusCode, socket[symbols.kSocketId], socket[symbols.kHandledResponses],
+        formatSocket(socket));
+    } else {
+      debug('[%s] Request#%d get %s response headers on Unknown Socket',
+        name, opaque[symbols.kRequestId], response.statusCode);
+    }
 
     if (!opaque[symbols.kEnableRequestTiming]) return;
     opaque[symbols.kRequestTiming].waiting = performanceTime(opaque[symbols.kRequestStartTime]);
@@ -197,7 +202,9 @@ export function initDiagnosticsChannel() {
   subscribe('undici:request:trailers', (message, name) => {
     const { request } = message as DiagnosticsChannel.RequestTrailersMessage;
     const opaque = getRequestOpaque(request, kHandler);
-    if (!opaque || !opaque[symbols.kRequestId]) return;
+    if (!opaque || !opaque[symbols.kRequestId]) {
+      return;
+    }
 
     debug('[%s] Request#%d get response body and trailers', name, opaque[symbols.kRequestId]);