Skip to content

Commit

Permalink
fix: Make sure the streaming process is killed on falure
Browse files Browse the repository at this point in the history
  • Loading branch information
mykola-mokhnach authored Sep 7, 2023
1 parent b529332 commit 605552e
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions lib/tools/video-commands.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,9 @@ videoCommands.startVideoStream = async function startVideoStream (opts) {
timeoutMs,
`The IDB video streamer has failed to start streaming after ${timeoutMs}ms timeout`
);
} catch (e) {
videoStreamProcess.kill();
throw e;
} finally {
videoStreamProcess.stderr.removeAllListeners('data');
videoStreamProcess.stdout.removeAllListeners('data');
Expand Down

0 comments on commit 605552e

Please sign in to comment.