nodejs
diff --git a/Collapse file
‎doc/api/repl.markdown‎
Copy file name to clipboardExpand all lines: doc/api/repl.markdown
+1-1Lines changed: 1 addition & 1 deletion
Display the source diff
Display the rich diff b/Collapse file
‎doc/api/repl.markdown‎
Copy file name to clipboardExpand all lines: doc/api/repl.markdown
+1-1Lines changed: 1 addition & 1 deletion
Display the source diff
Display the rich diff
diff --git a/Collapse file
‎lib/internal/repl.js‎
Copy file name to clipboardExpand all lines: lib/internal/repl.js
+12-1Lines changed: 12 additions & 1 deletion b/Collapse file
‎lib/internal/repl.js‎
Copy file name to clipboardExpand all lines: lib/internal/repl.js
+12-1Lines changed: 12 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-repl-persistent-history.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-repl-persistent-history.js
+5Lines changed: 5 additions & 0 deletions b/Collapse file
‎test/parallel/test-repl-persistent-history.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-repl-persistent-history.js
+5Lines changed: 5 additions & 0 deletions
@@ -37,7 +37,7 @@ via the following environment variables:
  - `NODE_REPL_HISTORY` - When a valid path is given, persistent REPL history
    will be saved to the specified file rather than `.node_repl_history` in the
    user's home directory. Setting this value to `""` will disable persistent
-   REPL history.
+   REPL history. Whitespace will be trimmed from the value.
  - `NODE_REPL_HISTORY_SIZE` - defaults to `1000`. Controls how many lines of
    history will be persisted if history is available. Must be a positive number.
  - `NODE_REPL_MODE` - may be any of `sloppy`, `strict`, or `magic`. Defaults
 
@@ -55,15 +55,26 @@ function createRepl(env, opts, cb) {
   }
 
   const repl = REPL.start(opts);
-  if (opts.terminal && env.NODE_REPL_HISTORY !== '') {
+  if (opts.terminal) {
     return setupHistory(repl, env.NODE_REPL_HISTORY,
                         env.NODE_REPL_HISTORY_FILE, cb);
   }
+
   repl._historyPrev = _replHistoryMessage;
   cb(null, repl);
 }
 
 function setupHistory(repl, historyPath, oldHistoryPath, ready) {
+  // Empty string disables persistent history.
+
+  if (typeof historyPath === 'string')
+    historyPath = historyPath.trim();
+
+  if (historyPath === '') {
+    repl._historyPrev = _replHistoryMessage;
+    return ready(null, repl);
+  }
+
   if (!historyPath) {
     try {
       historyPath = path.join(os.homedir(), '.node_repl_history');
 
@@ -85,6 +85,11 @@ const tests = [
     test: [UP],
     expected: [prompt, replDisabled, prompt]
   },
+  {
+    env: { NODE_REPL_HISTORY: ' ' },
+    test: [UP],
+    expected: [prompt, replDisabled, prompt]
+  },
   {
     env: { NODE_REPL_HISTORY: '',
            NODE_REPL_HISTORY_FILE: enoentHistoryPath },