Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stdout maxBuffer exceeded #2

Open
pixeldrew opened this issue Mar 29, 2018 · 5 comments
Open

stdout maxBuffer exceeded #2

pixeldrew opened this issue Mar 29, 2018 · 5 comments

Comments

@pixeldrew
Copy link

{ Error: stdout maxBuffer exceeded
at Socket.onChildStdout (child_process.js:258:14)
at emitOne (events.js:96:13)
at Socket.emit (events.js:191:7)
at readableAddChunk (_stream_readable.js:178:18)
at Socket.Readable.push (_stream_readable.js:136:10)
at Pipe.onread (net.js:561:20) cmd: 'npm publish --tag snapshot' }

npm version
{ npm: '5.7.1',
ares: '1.10.1-DEV',
cldr: '30.0.3',
http_parser: '2.7.0',
icu: '58.2',
modules: '51',
node: '7.10.1',
openssl: '1.0.2k',
tz: '2016j',
unicode: '9.0',
uv: '1.11.0',
v8: '5.5.372.43',
zlib: '1.2.11' }

Also, it'd be good to have the output of npm publish given as a log to stdout.

@pixeldrew
Copy link
Author

Suggest having an option to override the exec's default stdout maxBuffer Limit and redirect the publish commands to process.stdout/stderr

@keirlawson
Copy link
Contributor

Thanks for the bug report :) Are you able to share a snippet of code demonstrating the issue? or the output when you run an npm publish manually? I think having an option to override the buffer size is probably a good idea, but keen to see an example of what's causing the issue first.

@pixeldrew
Copy link
Author

pixeldrew commented Mar 30, 2018 via email

@pixeldrew
Copy link
Author

If I get some time I'll submit a pr

@keirlawson
Copy link
Contributor

Ok great, a PR would be fantastic. In the mean time I have increased the default buffer by 10x on master to make this issue less likely, but an option to set the buffer size and to display stdout would both be great additions

pixeldrew added a commit to pixeldrew/snapshot-publish that referenced this issue Apr 1, 2018
Added options to redirect output of npm publish commands to std out and also updated examples to show that stdout is returned.

Removed pify require
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants