From 61d92faf089a05ebeec2ec97565eae62aaed9a68 Mon Sep 17 00:00:00 2001
From: Marc Udoff <udoff@deshaw.com>
Date: Tue, 20 Sep 2016 18:21:44 -0400
Subject: [PATCH] Add NODE_PRESERVE_SYMLINKS environment variable

Add a way through environment variables to set the --preserve-symlinks
flag. Any non-null value of NODE_PRESERVE_SYMLINKS will enable symlinks.

Fixes: https://github.com/nodejs/node/issues/8509
---
 doc/api/cli.md                        |  7 +++++++
 src/node.cc                           |  5 +++++
 test/parallel/test-require-symlink.js | 12 +++++++++++-
 3 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index 221d0c631a26e9..4ab6cf1760ddc7 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -277,6 +277,13 @@ added: v0.11.15
 Data path for ICU (Intl object) data. Will extend linked-in data when compiled
 with small-icu support.
 
+### `NODE_PRESERVE_SYMLINKS=1`
+<!-- YAML
+added: REPLACEME
+-->
+
+When set to `1`, instructs the module loader to preserve symbolic links when
+resolving and caching modules.
 
 ### `NODE_REPL_HISTORY=file`
 <!-- YAML
diff --git a/src/node.cc b/src/node.cc
index be0c42b19f7b39..bbc1042f60b289 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -4188,6 +4188,11 @@ void Init(int* argc,
   V8::SetFlagsFromString(NODE_V8_OPTIONS, sizeof(NODE_V8_OPTIONS) - 1);
 #endif
 
+  // Allow for environment set preserving symlinks.
+  if (auto preserve_symlinks = secure_getenv("NODE_PRESERVE_SYMLINKS")) {
+    config_preserve_symlinks = (*preserve_symlinks == '1');
+  }
+
   // Parse a few arguments which are specific to Node.
   int v8_argc;
   const char** v8_argv;
diff --git a/test/parallel/test-require-symlink.js b/test/parallel/test-require-symlink.js
index 38139ef1cfc57d..0792bc13bb70a7 100644
--- a/test/parallel/test-require-symlink.js
+++ b/test/parallel/test-require-symlink.js
@@ -6,6 +6,7 @@ const path = require('path');
 const fs = require('fs');
 const exec = require('child_process').exec;
 const spawn = require('child_process').spawn;
+const util = require('util');
 
 common.refreshTmpDir();
 
@@ -53,7 +54,16 @@ function test() {
   const node = process.execPath;
   const child = spawn(node, ['--preserve-symlinks', linkScript]);
   child.on('close', function(code, signal) {
-    assert(!code);
+    assert.strictEqual(code, 0);
+    assert(!signal);
+  });
+
+  // Also verify that symlinks works for setting preserve via env variables
+  const childEnv = spawn(node, [linkScript], {
+    env: util._extend(process.env, {NODE_PRESERVE_SYMLINKS: '1'})
+  });
+  childEnv.on('close', function(code, signal) {
+    assert.strictEqual(code, 0);
     assert(!signal);
   });
 }