fixup: fix util.format on a revoked proxy

Signed-off-by: Daeyeon Jeong [email protected]
nodejs · May 15, 2022 · ddf91e4 · ddf91e4
1 parent 5723aaf
commit ddf91e4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
@@ -1979,7 +1979,7 @@ function hasBuiltInToString(value) {
   }
 
   // Count objects that have no `toString` function as built-in.
-  if (typeof value.toString !== 'function') {
+  if (typeof value?.toString !== 'function') {
     return true;
   }
 

diff --git a/test/parallel/test-util-inspect-proxy.js b/test/parallel/test-util-inspect-proxy.js
@@ -76,6 +76,8 @@ assert.strictEqual(
   '{ proxy: Proxy [ null, null ], revoke: [Function (anonymous)] }',
 );
 
+assert.strictEqual(util.format('%s', r.proxy), 'Proxy []');
+
 assert.strictEqual(
   util.inspect(proxyObj, opts),
   'Proxy [\n' +