stream: add custom inspect to BufferList

Currently inspecting the BufferList can result a maximum call stack
size error. Adding a individual inspect function prevents this.

PR-URL: https://github.com/nodejs/node/pull/17907
Refs: https://github.com/nodejs/node/issues/12693
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

1 files changed, 6 insertions, 0 deletions

diff --git a/lib/internal/streams/BufferList.js b/lib/internal/streams/BufferList.js
index 23d5a8a2db..c5a293b9ca 100644
--- a/lib/internal/streams/BufferList.js
+++ b/lib/internal/streams/BufferList.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const { Buffer } = require('buffer');
+const { inspect } = require('util');
 
 function copyBuffer(src, target, offset) {
   Buffer.prototype.copy.call(src, target, offset);
@@ -73,4 +74,9 @@ module.exports = class BufferList {
     }
     return ret;
   }
+
+  [inspect.custom]() {
+    const obj = inspect({ length: this.length });
+    return `${this.constructor.name} ${obj}`;
+  }
 };