Child Process send incorrectly serializes values

[jacobp100]

Code

index.js

const childProcess = require('child_process').fork(__dirname + '/worker.js');
childProcess.send('get-infinity');
childProcess.on('message', (valuePair) =>{
 console.log(valuePair);
});

worker.js

process.on('message', () =>{
 process.send(['Infinity', Infinity]);
 process.send(['NaN', NaN]);
});

Expected Result

['Infinity', Infinity]
['NaN', NaN]

Actual Result

['Infinity', null]
['NaN', null]

• Version: _output of node -v

v5.6.0

• Platform: _either uname -a output, or if Windows, version and 32-bit or
  64-bit

Linux tido-tchaikovsky 3.19.0-51-generic #57~14.04.1-Ubuntu SMP Fri Feb 19 14:36:55 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

• Subsystem: optional. if known - please specify affected core module name

child_process

[cjihrig]

The message is stringified prior to sending using JSON.stringify():

> JSON.stringify(NaN);
'null'
> JSON.stringify(Infinity);
'null'

[evanlucas]

This is actually according to the specification. Please see http://www.ecma-international.org/ecma-262/6.0/#sec-json.stringify, specifically Note 4.

NOTE 4 Finite numbers are stringified as if by calling ToString(number). NaN and Infinity regardless of sign are represented as the String null.

Closing as this is working as designed. Thanks!

[jacobp100]

I know it's according to the json spec. But i don't think this issue should be closed. Firstly, it does not specify in the documentation for process that it uses json stringify for data transfer. Secondly, there are other ways to serialise data that wouldn't suffer from this.

[evanlucas]

@jacobp100 Fair enough. At this point, changing the way serialization between processes is done would be a pretty large undertaking. I do recall that it has been considered though. For now, I think that a doc update would suffice. Would you be willing to submit a Pull Request with that update? Thanks!

[bnoordhuis]

The documentation does mention in a few places that JSON is used but I agree it could be more explicit, particularly the reference documentation for process.send() and ChildProcess#send().

Secondly, there are other ways to serialise data that wouldn't suffer from this.

Sure, but we're not using that, have never used that in the past and I don't think this particular issue is important enough to merit writing a high-performance serializer/deserializer from scratch. That's without even getting into the issue of backwards compatibility.

[abenhamdine]

@jacobp100 do you plan to submit a PR for a doc update ? If no, I'm volunteer to do it as I'm interested in this.

[jacobp100]

Go for it!