vmadm stdin read limit

[dn0]

Hello,

vmadm and probably other nodejs command line tools are affected by a nodejs bug shipped in SmartOS. Passing a valid json via stdin will lead to an error when the json payload is too large (in my case larger than 20481 bytes):

json --validate -f test.json; echo $?
0
cat test.json | vmadm update <uuid>
Invalid JSON payload: Unexpected end of input

The problem is in fs.readFile() and mentioned here: nodejs/node-v0.x-archive#7412 (fixed here: nodejs/node-v0.x-archive@a6af709)

[dn0]

I realized that this should go into illumos-extra. I made and tested a patch: https://github.com/joyent/illumos-extra/compare/master...erigones:master?expand=1

[bahamat]

@dn0 The best way to fix this is for vmadm to read process.stdin rather than patching node itself. I've filed OS-6756 internally for this.

[dn0]

@bahamat Thanks