util: fix nested proxy inspection

Fixes: #61061

Author: BridgeAR

lib/internal/util/inspect.js

@@ -173,6 +173,7 @@ const {
   isModuleNamespaceObject,
   isNativeError,
   isPromise,
+  isProxy,
   isSet,
   isSetIterator,
   isWeakMap,
@@ -1122,13 +1123,19 @@ function formatValue(ctx, value, recurseTimes, typedArray) {
   // any proxy handlers.
   const proxy = getProxyDetails(value, !!ctx.showProxy);
   if (proxy !== undefined) {
-    if (proxy === null || proxy[0] === null) {
-      return ctx.stylize('<Revoked Proxy>', 'special');
-    }
     if (ctx.showProxy) {
+      if (proxy[0] === null) {
+        return ctx.stylize('<Revoked Proxy>', 'special');
+      }
       return formatProxy(ctx, proxy, recurseTimes);
     }
+    if (proxy === null) {
+      return ctx.stylize('<Revoked Proxy>', 'special');
+    }
     value = proxy;
+    if (isProxy(value)) {
+      return formatValue(ctx, value, recurseTimes);
+    }
   }
 
   // Provide a hook for user-specified inspect functions.

test/parallel/test-util-inspect-proxy.js

@@ -179,3 +179,12 @@ const expected10 = '[Function (anonymous)]';
 const expected11 = '[Function (anonymous)]';
 assert.strictEqual(util.inspect(proxy10), expected10);
 assert.strictEqual(util.inspect(proxy11), expected11);
+
+{
+  // Nested proxies should not trigger any proxy handlers.
+  const errorProxy = new Proxy({}, { get(...cause) { throw new EvalError('boom', { cause }); } });
+  const nestedProxy = new Proxy(errorProxy, {});
+
+  util.inspect(nestedProxy, { showProxy: true });
+  util.inspect(nestedProxy, { showProxy: false });
+}