ffprobe throws error for portrait videos on Windows

var def = deferred();
ffmpeg.ffprobe(zpath, function (err, metadata) {
     if (err) return def.promise.emit('error', err);
     def.promise.emit('done', metadata);
});
return def.promise;

This works for landscape videos but fails for portrait videos.

Error:
[TypeError: Object.keys called on non-object]

This also breaks 'screenshots' method as it uses ffprobe.

Node Verison: "fluent-ffmpeg": "^2.0.1",

FFMPEG Version:
This is a FFmpeg Win64 static build by Kyle Schwarz.
Zeranoe's FFmpeg Builds Home Page: http://ffmpeg.zeranoe.com/builds/
FFmpeg version: 2015-05-21 git-0b9d636

Is this a Windows only issue?

[njoyard]

Could you please give the error stack trace and ffprobe command line output for the failing video ?

[nickwhiteley]

I have this problem too. A stack trace is included below. data.format is undefined and data.streams is populated on line 152 of ffprobe.js and it's the undefined data.format which is causing the problem.

Moving from the pre-compiled FFmpeg Windows build "2015-04-28 git-b410c69" to "2015-07-02 git-03b2b40" caused this for me (changing back fixes it).
The latest pre-compiled Ubuntu Linux version appears to have the same problem.

If I run " ffprobe -show_format abc.mp4" on the command line against the old Windows and new Windows version it looks same when I scan it by eye, version numbers aside.

This only affects videos in portrait and I've only tried it with uploads from one device.

Stack trace:

e:\Mercurial\nodejs\iflicks\node_modules\fluent-ffmpeg\lib\ffprobe.js:153
var legacyTagKeys = Object.keys(target).filter(legacyTag);
^
TypeError: Object.keys called on non-object
at Function.keys (native)
at e:\Mercurial\nodejs\iflicks\node_modules\fluent-ffmpeg\lib\ffprobe.js:153:42
at Array.forEach (native)
at handleExit (e:\Mercurial\nodejs\iflicks\node_modules\fluent-ffmpeg\lib\ffprobe.js:152:46)
at ChildProcess. (e:\Mercurial\nodejs\iflicks\node_modules\fluent-ffmpeg\lib\ffprobe.js:191:11)
at ChildProcess.emit (events.js:110:17)
at Process.ChildProcess._handle.onexit (child_process.js:1074:12)
6 Jul 16:07:44 - [nodemon] app crashed - waiting for file changes before starting...

[nickwhiteley]

The problem is with an empty line returned by ffprobe in a new section called [SIDE_DATA]. The empty line causes one of the while loops to drop out.
Adding "lines = lines.filter(Boolean);" to line 15 of /lib/ffprobe.js removes blank entries and fixes it.

[jdp]

I'm seeing this issue too. The fix from @nickwhiteley works for me as well.

[njoyard]

Will be fixed by #412, which I'll merge ASAP.

[brunosiqueira]

@nickwhiteley 's fix worked for me to! Is there a planed release with it?

[ashokfernandez]

+1

confirmed.. master is working for both portrait and landscape on Windows using same FFMPEG Version as my first comment

@nickwhiteley apologies for dropping the ball on error trace and thank you for following up and debugging through it
@njoyard great work on this lib.. it is fantastic and really simple to work with .. {"thank":"you"}

"dependencies": {
    "fluent-ffmpeg": "git://github.com/fluent-ffmpeg/node-fluent-ffmpeg.git#master"
  }