search

madebyhiro / codem-transcode

Offline video transcoder written in node.js
Other
153 stars 68 forks source link

ffmpeg stderr buffer size #59

Closed bartzon closed 8 years ago

bartzon commented 8 years ago

Looks like ffmpeg can report more on stderr than node's maxBuffer is expecting. See http://stackoverflow.com/questions/23429499/stdout-buffer-issue-using-node-child-process for a similar type of error. This causes the ffmpeg process to finish with an error, and therefore the transcoding fails.

Could you take a look at this? Thanks!

tieleman commented 8 years ago

This commit (https://github.com/madebyhiro/codem-transcode/commit/eb6114fe49374a550ad8570392482b30f2ecde2b) already increases the buffer to 4MB. Not enough? We could increase it.