add more logs

app/main.py

@@ -17,21 +17,16 @@ def health_check():
     return {"status": "ok", "message": "YouTube Downloader API is running."}
 
 @app.post("/download")
-async def download(payload: DownloadRequest):  # ✅ tên đúng
+async def download(payload: DownloadRequest):
     try:
-        video_path = download_video(payload.url, payload.cookie)
+        video_id = str(uuid.uuid4())
+        video_path = download_video(payload.url, payload.cookie, video_id=video_id)
 
-        filename = os.path.basename(video_path)
-        return FileResponse(
-            path=video_path,
-            filename=filename,
-            media_type="video/mp4"
-        )
+        return {"download_url": f"/video/{video_id}.mp4"}
     except Exception as e:
         print("🔥 API download error:", str(e))
         raise HTTPException(status_code=500, detail=str(e))
 
-
 @app.get("/video/{video_id}")
 def get_video(video_id: str, background_tasks: BackgroundTasks):
     filepath = f"/tmp/{video_id}.mp4"

app/utils.py

@@ -2,9 +2,8 @@ import yt_dlp
 import tempfile
 import os
 
-def download_video(url: str, cookie: str = None) -> str:
-    safe_filename = "video"
-    output_path = f"/tmp/{safe_filename}.mp4"
+def download_video(url: str, cookie: str = None, video_id: str = "video") -> str:
+    output_path = f"/tmp/{video_id}.mp4"
 
     ydl_opts = {
         "format": "best[ext=mp4]/best",
@@ -13,12 +12,22 @@ def download_video(url: str, cookie: str = None) -> str:
         "noplaylist": True,
     }
 
+    cookie_file = None
+
     if cookie:
-        # Save cookie to a temp file
-        with tempfile.NamedTemporaryFile(delete=False, mode="w") as f:
-            f.write(cookie)
-            cookie_file = f.name
-        ydl_opts["cookiefile"] = cookie_file
+        # Convert raw cookie string to Netscape format
+        try:
+            cookie_file = tempfile.NamedTemporaryFile(delete=False, mode="w", suffix=".txt")
+            cookie_file.write("# Netscape HTTP Cookie File\n")
+            for pair in cookie.split(";"):
+                parts = pair.strip().split("=")
+                if len(parts) == 2:
+                    name, value = parts
+                    cookie_file.write(f".youtube.com\tTRUE\t/\tFALSE\t2145916800\t{name}\t{value}\n")
+            cookie_file.close()
+            ydl_opts["cookiefile"] = cookie_file.name
+        except Exception as e:
+            print("🔥 Failed to process cookies:", str(e))
 
     try:
         print(f"📥 Downloading video from {url} to {output_path}")
@@ -28,13 +37,12 @@ def download_video(url: str, cookie: str = None) -> str:
         if not os.path.exists(output_path):
             raise RuntimeError("Download finished but output file not found.")
 
-        print(f"✅ Video downloaded successfully: {output_path}")
         return output_path
 
     except Exception as e:
         print("🔥 Download failed:", str(e))
         raise
     finally:
-        if cookie and 'cookie_file' in locals() and os.path.exists(cookie_file):
-            os.remove(cookie_file)
+        if cookie_file and os.path.exists(cookie_file.name):
+            os.remove(cookie_file.name)
             print("🧹 Temporary cookie file removed.")