Killing the ffmpeg process if it will not cancel within 1 second

Since the cancellation token does not  stop the Stream.ReadAsync when the inputsource is offline,  we manually kill the ffmpeg process after 1 second of lenience. This causes the ReadAsync method to return 0 bytes, allowing the stream to end.

Author: JPasterkampRotec

src/Nager.VideoStream/VideoStreamClient.cs

@@ -54,6 +54,7 @@ public async Task StartFrameReaderAsync(
             };
 
             using (var ffmpegProcess = new Process { StartInfo = startInfo })
+            using (cancellationToken.Register(() => WaitBeforeKill(ffmpegProcess)))
             {
                 ffmpegProcess.ErrorDataReceived += this.ProcessDataReceived;
 
@@ -99,18 +100,23 @@ public async Task StartFrameReaderAsync(
 
                 ffmpegProcess.ErrorDataReceived -= this.ProcessDataReceived;
 
-                ffmpegProcess.WaitForExit(1000);
-
-                if (!ffmpegProcess.HasExited)
-                {
-                    ffmpegProcess.Kill();
-                }
+                WaitBeforeKill(ffmpegProcess);
             }
         }
 
         private void ProcessDataReceived(object sender, DataReceivedEventArgs e)
         {
             this.FFmpegInfoReceived?.Invoke(e.Data);
         }
+
+        private static void WaitBeforeKill(Process process, int waitForExitMilliseconds = 1000)
+        {
+            process.WaitForExit(waitForExitMilliseconds);
+
+            if (!process.HasExited)
+            {
+                process.Kill();
+            }
+        }
     }
 }