tty: enable buffering

ronag · ronag · commit a8a86387fa4a · 2021-07-07T14:50:47.000+02:00
HWM was set to 0 which would cause e.g. stdout.write(...) to always return false. Refs: #39246 PR-URL: #39253 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
diff --git a/lib/tty.js b/lib/tty.js
@@ -57,7 +57,7 @@ function ReadStream(fd, options) {
   }
 
   net.Socket.call(this, {
-    highWaterMark: 0,
+    readableHighWaterMark: 0,
     handle: tty,
     manualStart: true,
     ...options
@@ -94,7 +94,7 @@ function WriteStream(fd) {
   }
 
   net.Socket.call(this, {
-    highWaterMark: 0,
+    readableHighWaterMark: 0,
     handle: tty,
     manualStart: true
   });
diff --git a/test/parallel/test-stdout-stderr-write.js b/test/parallel/test-stdout-stderr-write.js
@@ -0,0 +1,8 @@
+'use strict';
+
+require('../common');
+const assert = require('assert');
+
+// https://github.com/nodejs/node/pull/39246
+assert.strictEqual(process.stderr.write('asd'), true);
+assert.strictEqual(process.stdout.write('asd'), true);
