diff options
author | Rich Trott <rtrott@gmail.com> | 2018-12-23 21:59:19 -0800 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2018-12-26 14:18:55 +0100 |
commit | 79aab5dd7cfe30f807b5d197c95aea9dd59ecb40 (patch) | |
tree | 58606b54359b62f4830c57cd56ae33fa43109e18 | |
parent | 86e2ec41af072248b88629df3dccbd4d557fab20 (diff) | |
download | android-node-v8-79aab5dd7cfe30f807b5d197c95aea9dd59ecb40.tar.gz android-node-v8-79aab5dd7cfe30f807b5d197c95aea9dd59ecb40.tar.bz2 android-node-v8-79aab5dd7cfe30f807b5d197c95aea9dd59ecb40.zip |
net: use decodeStrings public API for writable stream
Instead of using an undocumented underscore-prefixed property to
configure the writable stream instance to not encode strings as buffers,
use the public API which is an options property passed to the
constructor.
Refs: https://github.com/nodejs/node/issues/445
PR-URL: https://github.com/nodejs/node/pull/25201
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
-rw-r--r-- | lib/net.js | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/net.js b/lib/net.js index 0577dd3c9f..7680c8860e 100644 --- a/lib/net.js +++ b/lib/net.js @@ -255,6 +255,8 @@ function Socket(options) { options.allowHalfOpen = true; // For backwards compat do not emit close on destroy. options.emitClose = false; + // Handle strings directly. + options.decodeStrings = false; stream.Duplex.call(this, options); // Default to *not* allowing half open sockets. @@ -308,9 +310,6 @@ function Socket(options) { this._pendingData = null; this._pendingEncoding = ''; - // handle strings directly - this._writableState.decodeStrings = false; - // If we have a handle, then start the flow of data into the // buffer. if not, then this will happen when we connect if (this._handle && options.readable !== false) { |