From 6a701ee8c064a4652db475c93ee89c3ffe112986 Mon Sep 17 00:00:00 2001
From: Shubbu03 <thatcoderguyshubham@gmail.com>
Date: Mon, 23 Jun 2025 22:22:43 +0530
Subject: [PATCH] better error handling

---
 apps/web/src/lib/ffmpeg-utils.ts | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/apps/web/src/lib/ffmpeg-utils.ts b/apps/web/src/lib/ffmpeg-utils.ts
index de203fd..ae0c267 100644
--- a/apps/web/src/lib/ffmpeg-utils.ts
+++ b/apps/web/src/lib/ffmpeg-utils.ts
@@ -108,18 +108,26 @@ export const getVideoInfo = async (videoFile: File): Promise<{
   // Write input file
   await ffmpeg.writeFile(inputName, new Uint8Array(await videoFile.arrayBuffer()));
 
-  // Capture FFmpeg stderr output
+  // Capture FFmpeg stderr output with a one-time listener pattern
   let ffmpegOutput = '';
+  let listening = true;
   const listener = (data: string) => {
-    ffmpegOutput += data;
+    if (listening) ffmpegOutput += data;
   };
   ffmpeg.on('log', ({ message }) => listener(message));
 
   // Run ffmpeg to get info (stderr will contain the info)
-  await ffmpeg.exec(['-i', inputName, '-f', 'null', '-']);
+  try {
+    await ffmpeg.exec(['-i', inputName, '-f', 'null', '-']);
+  } catch (error) {
+    listening = false;
+    await ffmpeg.deleteFile(inputName);
+    console.error('FFmpeg execution failed:', error);
+    throw new Error('Failed to extract video info. The file may be corrupted or in an unsupported format.');
+  }
 
-  // Remove listener
-  // (No off() method in ffmpeg.wasm, so this is a no-op, but included for clarity)
+  // Disable listener after exec completes
+  listening = false;
 
   // Cleanup
   await ffmpeg.deleteFile(inputName);
@@ -220,4 +228,4 @@ export const extractAudio = async (
   await ffmpeg.deleteFile(outputName);
   
   return blob;
-}; 
\ No newline at end of file
+};
\ No newline at end of file