src: allow getting Symbols on process.env

1aa595e5bd6 introduced a `throw` for accessing `Symbol` properties of
`process.env`. However, this breaks `util.inspect(process)` and
things like `Object.prototype.toString.call(process.env)`, so this
patch changes the behaviour for the getter to just always return
`undefined`.

Ref: https://github.com/nodejs/node/pull/9446
Fixes: https://github.com/nodejs/node/issues/9641
PR-URL: https://github.com/nodejs/node/pull/9631
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

1 files changed, 5 insertions, 4 deletions

diff --git a/test/parallel/test-process-env-symbols.js b/test/parallel/test-process-env-symbols.js
index bb6e1f8778..a8798fc457 100644
--- a/test/parallel/test-process-env-symbols.js
+++ b/test/parallel/test-process-env-symbols.js
@@ -5,10 +5,8 @@ const assert = require('assert');
 const symbol = Symbol('sym');
 const errRegExp = /^TypeError: Cannot convert a Symbol value to a string$/;
 
-// Verify that getting via a symbol key throws.
-assert.throws(() => {
-  process.env[symbol];
-}, errRegExp);
+// Verify that getting via a symbol key returns undefined.
+assert.strictEqual(process.env[symbol], undefined);
 
 // Verify that assigning via a symbol key throws.
 assert.throws(() => {
@@ -24,3 +22,6 @@ assert.throws(() => {
 assert.throws(() => {
   symbol in process.env;
 }, errRegExp);
+
+// Checks that well-known symbols like `Symbol.toStringTag` won’t throw.
+assert.doesNotThrow(() => Object.prototype.toString.call(process.env));