Merge pull request #785 from HaveAGitGat/streams

feat: update error message when no stream data

FlowPlugins/CommunityFlowPlugins/ffmpegCommand/ffmpegCommandStart/1.0.0/index.js

@@ -43,10 +43,19 @@ var plugin = function (args) {
     // eslint-disable-next-line @typescript-eslint/no-unused-vars,no-param-reassign
     args.inputs = lib.loadDefaultValues(args.inputs, details);
     var container = (0, fileUtils_1.getContainer)(args.inputFileObj._id);
+    var streams = [];
+    try {
+        streams = JSON.parse(JSON.stringify(args.inputFileObj.ffProbeData.streams));
+    }
+    catch (err) {
+        var message = "Error parsing FFprobe streams, it seems FFprobe could not scan the file: ".concat(JSON.stringify(err));
+        args.jobLog(message);
+        throw new Error(message);
+    }
     var ffmpegCommand = {
         init: true,
         inputFiles: [],
-        streams: JSON.parse(JSON.stringify(args.inputFileObj.ffProbeData.streams)).map(function (stream) { return (__assign(__assign({}, stream), { removed: false, mapArgs: [
+        streams: streams.map(function (stream) { return (__assign(__assign({}, stream), { removed: false, mapArgs: [
                 '-map',
                 "0:".concat(stream.index),
             ], inputArgs: [], outputArgs: [] })); }),

FlowPluginsTs/CommunityFlowPlugins/ffmpegCommand/ffmpegCommandStart/1.0.0/index.ts

@@ -39,10 +39,20 @@ const plugin = (args:IpluginInputArgs):IpluginOutputArgs => {
 
   const container = getContainer(args.inputFileObj._id);
 
+  let streams = [];
+
+  try {
+    streams = JSON.parse(JSON.stringify(args.inputFileObj.ffProbeData.streams));
+  } catch (err) {
+    const message = `Error parsing FFprobe streams, it seems FFprobe could not scan the file: ${JSON.stringify(err)}`;
+    args.jobLog(message);
+    throw new Error(message);
+  }
+
   const ffmpegCommand = {
     init: true,
     inputFiles: [],
-    streams: JSON.parse(JSON.stringify(args.inputFileObj.ffProbeData.streams)).map((stream:Istreams) => ({
+    streams: streams.map((stream:Istreams) => ({
       ...stream,
       removed: false,
       mapArgs: [