Update app.py

app.py

@@ -122,77 +122,77 @@ def is_youtube_url(url):
 
 
 def download_youtube_video(url):
-    """Télécharge une vidéo YouTube en utilisant subprocess avec yt-dlp"""
+    """Télécharge une vidéo YouTube en utilisant yt-dlp avec une configuration adaptée aux conteneurs"""
     try:
         with tempfile.TemporaryDirectory() as temp_dir:
             output_template = os.path.join(temp_dir, '%(title)s.%(ext)s')
             
-            # Enhanced configuration for yt-dlp
+            # Configuration optimisée pour environnement conteneurisé
             command = [
                 'yt-dlp',
                 '--format', 'best[filesize<50M]',
                 '--quiet',
                 '--no-warnings',
                 '--output', output_template,
-                '--cookies-from-browser', 'chrome',  # Use Chrome cookies to appear as logged-in user
-                '--user-agent', 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36',  # Modern user agent
-                '--sleep-interval', '2',  # Add small delay between requests
-                '--max-sleep-interval', '5',
-                '--socket-timeout', '10',
-                '--retries', '3',
+                '--extract-audio',  # Tenter d'abord l'extraction audio si la vidéo échoue
+                '--format-sort', 'res,ext:mp4:m4a',  # Prioriser les formats plus légers
+                '--user-agent', 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36',
+                '--referer', 'https://www.youtube.com/',
+                '--geo-bypass',  # Contourner les restrictions géographiques
+                '--no-check-certificates',  # Ignorer les erreurs de certificat
+                '--force-ipv4',  # Forcer IPv4 pour plus de stabilité
+                '--ignore-errors',  # Continuer en cas d'erreurs mineures
+                '--no-playlist',  # Éviter le téléchargement de playlists
+                '--extractor-retries', '3',
+                '--file-access-retries', '3',
+                '--fragment-retries', '3',
+                '--retry-sleep', '5',
                 url
             ]
             
             logger.info(f"Début du téléchargement de la vidéo: {url}")
             
-            # Check if yt-dlp is installed
             try:
-                subprocess.run(['yt-dlp', '--version'], capture_output=True, check=True)
-            except (subprocess.SubprocessError, FileNotFoundError):
-                logger.error("yt-dlp n'est pas installé")
-                raise RuntimeError("yt-dlp n'est pas installé. Installez-le avec 'pip install yt-dlp'")
+                # Vérifier la version de yt-dlp
+                version_process = subprocess.run(['yt-dlp', '--version'], 
+                                              capture_output=True, 
+                                              text=True, 
+                                              check=True)
+                logger.info(f"Version de yt-dlp: {version_process.stdout.strip()}")
+            except subprocess.SubprocessError as e:
+                logger.error(f"Erreur lors de la vérification de yt-dlp: {e}")
+                raise RuntimeError("yt-dlp n'est pas correctement installé")
             
-            process = subprocess.Popen(
+            # Premier essai avec la configuration standard
+            process = subprocess.run(
                 command,
-                stdout=subprocess.PIPE,
-                stderr=subprocess.PIPE,
-                universal_newlines=True
+                capture_output=True,
+                text=True
             )
             
-            stdout, stderr = process.communicate()
-            
             if process.returncode != 0:
-                if "Sign in to confirm you're not a bot" in stderr:
-                    logger.error("Détection anti-bot YouTube - tentative avec configuration alternative")
-                    # Fallback approach with different configuration
-                    command = [
-                        'yt-dlp',
-                        '--format', 'best[filesize<50M]',
-                        '--quiet',
-                        '--no-warnings',
-                        '--output', output_template,
-                        '--cookies', 'youtube.com_cookies.txt',  # Use cookies file if available
-                        '--user-agent', 'Mozilla/5.0 (X11; Linux x86_64; rv:120.0) Gecko/20100101 Firefox/120.0',  # Alternative user agent
-                        '--sleep-interval', '3',
-                        '--max-sleep-interval', '6',
-                        '--socket-timeout', '15',
-                        '--retries', '5',
-                        '--no-check-certificates',  # Skip HTTPS certificate validation
-                        url
-                    ]
-                    process = subprocess.Popen(
-                        command,
-                        stdout=subprocess.PIPE,
-                        stderr=subprocess.PIPE,
-                        universal_newlines=True
-                    )
-                    stdout, stderr = process.communicate()
-                    if process.returncode != 0:
-                        logger.error(f"Erreur yt-dlp: {stderr}")
-                        raise Exception(f"Erreur yt-dlp: {stderr}")
-                else:
-                    logger.error(f"Erreur yt-dlp: {stderr}")
-                    raise Exception(f"Erreur yt-dlp: {stderr}")
+                logger.warning(f"Premier essai échoué: {process.stderr}")
+                logger.info("Tentative avec configuration alternative...")
+                
+                # Configuration alternative
+                alternative_command = command.copy()
+                alternative_command.extend([
+                    '--format', 'worst',  # Utiliser la qualité la plus basse
+                    '--prefer-insecure',  # Préférer les connexions non-sécurisées si nécessaire
+                    '--socket-timeout', '30',
+                    '--external-downloader', 'aria2c',  # Utiliser aria2c comme downloader alternatif
+                    '--external-downloader-args', 'aria2c:"--min-split-size=1M --max-connection-per-server=16 --max-concurrent-downloads=16 --split=16"'
+                ])
+                
+                process = subprocess.run(
+                    alternative_command,
+                    capture_output=True,
+                    text=True
+                )
+                
+                if process.returncode != 0:
+                    logger.error(f"Échec du téléchargement après tentatives alternatives: {process.stderr}")
+                    raise Exception(f"Échec du téléchargement: {process.stderr}")
             
             downloaded_files = os.listdir(temp_dir)
             if not downloaded_files:
@@ -201,6 +201,14 @@ def download_youtube_video(url):
             
             video_path = os.path.join(temp_dir, downloaded_files[0])
             
+            # Vérifier la taille du fichier
+            file_size = os.path.getsize(video_path)
+            if file_size == 0:
+                raise ValueError("Le fichier téléchargé est vide")
+            
+            logger.info(f"Fichier téléchargé: {video_path} ({file_size} bytes)")
+            
+            # Copier le fichier vers un emplacement temporaire
             temp_file = tempfile.NamedTemporaryFile(delete=False, suffix=os.path.splitext(video_path)[1])
             with open(video_path, 'rb') as f:
                 temp_file.write(f.read())