Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

stdout maxBuffer exceeded #126

Closed
macdja38 opened this issue Oct 2, 2016 · 9 comments
Closed

stdout maxBuffer exceeded #126

macdja38 opened this issue Oct 2, 2016 · 9 comments

Comments

@macdja38
Copy link

macdja38 commented Oct 2, 2016

{ Error: stdout maxBuffer exceeded
pvpcraft-1     at Socket.<anonymous> (child_process.js:259:14)
pvpcraft-1     at emitOne (events.js:96:13)
pvpcraft-1     at Socket.emit (events.js:188:7)
pvpcraft-1     at readableAddChunk (_stream_readable.js:177:18)
pvpcraft-1     at Socket.Readable.push (_stream_readable.js:135:10)
pvpcraft-1     at Pipe.onread (net.js:542:20)
pvpcraft-1   cmd: '/docker/pvpcraft/pvpcraft/node_modules/youtube-dl/bin/youtube-dl --dump-json -f best http://www.youtube.com/watch?v=VrD5JrW67t8' }

issue cased by calling

youtube-dl.getInfo("http://www.youtube.com/watch?v=VrD5JrW67t8", [], (error)=>{
  console.log(error);
}

where youtube-dl is the object provided by requiring this library

@macdja38
Copy link
Author

macdja38 commented Oct 2, 2016

After looking through the source I found the options object was passed through allowing the user to specify a maxBuffer size with

youtube-dl.getInfo("url", [], {maxBuffer: 1000*1024}, ()=>{})

I'm going to leave this open as I didn't see it in the readme or info example.

@EstebanFuentealba
Copy link

yes,
youtubedl("url" , ['--format=18'], { cwd: __dirname, maxBuffer: 1000*1024 });

@macdja38
Copy link
Author

macdja38 commented Oct 5, 2016

@EstebanFuentealba i didn't see that in the relevant example or readme, where was it?

@EstebanFuentealba
Copy link

not mentioned, i read the code

@macdja38
Copy link
Author

macdja38 commented Oct 5, 2016

@EstebanFuentealba Ahh... Yea I mentioned that it is possible to specify the maxBuffer in the commend directly above that, and that I'm leaving this open as it's not documented while being necessary to fetch info for some you-tube videos.

@wwooww
Copy link

wwooww commented Oct 7, 2016

I recommend maxBuffer: Infinity

@Mochlate
Copy link

Mochlate commented Jan 5, 2017

What is the drawback to specifying maxBuffer infinity?

@wwooww
Copy link

wwooww commented Jan 10, 2017

@Mochlate Umm.. I has not faced yet...

@macdja38
Copy link
Author

Fixed in 3bb3e4d

pietrop added a commit to pietrop/electron-video-downloader that referenced this issue Jul 7, 2018
youtube-dl error, by increasing size to Infinity as sudgested in this issue przemyslawpluta/node-youtube-dl#126
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants