Skip to content

Commit 7f44db9

Browse files
author
Ben Newman
committed
Revert "Work around a Node 8.1.0 regression that broke meteor login."
This reverts commit dd11432. This bug that necessitated this workaround was fixed by nodejs/node#13560, as mentioned in https://nodejs.org/en/blog/release/v8.1.1/.
1 parent 3b2899d commit 7f44db9

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

tools/console/console.js

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1325,15 +1325,6 @@ Console.prototype.readLine = function (options) {
13251325
var previousProgressDisplay = self._progressDisplay;
13261326
self._setProgressDisplay(new ProgressDisplayNone());
13271327

1328-
Object.getOwnPropertySymbols(process.stdin).forEach(sym => {
1329-
if (/^Symbol\(\w+-decoder\)$/.test(String(sym))) {
1330-
// Node 8.1.0 fails to attach a 'data' listener to process.stdin if
1331-
// it already has a StringDecoder property, so we have to remove
1332-
// that property if we want to use process.stdin again. #node8
1333-
delete process.stdin[sym];
1334-
}
1335-
});
1336-
13371328
// Read a line, throwing away the echoed characters into our dummy stream.
13381329
var rl = readline.createInterface({
13391330
input: process.stdin,

0 commit comments

Comments
 (0)