deployment warnings

video_downloader.py

@@ -19,7 +19,7 @@ class VideoDownloader:
         self.output_dir = Path(output_dir)
         self.output_dir.mkdir(exist_ok=True)
         
-        # yt-dlp options with automatic cookie support
+        # yt-dlp options with simplified cookie support
         self.ydl_opts = {
             'outtmpl': str(self.output_dir / '%(title)s.%(ext)s'),
             'format': 'best',
@@ -29,42 +29,62 @@ class VideoDownloader:
             'extract_flat': False,
         }
         
-        # Add automatic browser cookie support
+        # Configure cookies based on environment
         self._configure_cookies()
     
     def _configure_cookies(self):
-        """Configure automatic cookie extraction from browser"""
+        """Configure cookie extraction - simplified approach"""
         try:
-            # Try different browsers in order of preference
-            browsers_to_try = ['chrome', 'firefox', 'edge', 'safari', 'opera', 'brave']
-            
-            for browser in browsers_to_try:
+            # Check if we're in a deployment environment (no display, limited file system)
+            is_deployment = (
+                os.getenv('SPACE_ID') is not None or  # Hugging Face Spaces
+                os.getenv('RAILWAY_ENVIRONMENT') is not None or  # Railway
+                os.getenv('RENDER') is not None or  # Render
+                not os.getenv('DISPLAY', '')  # No display environment
+            )
+            
+            if is_deployment:
+                logger.info("Deployment environment detected - skipping browser cookies")
+                self._configure_deployment_options()
+                return
+            
+            # Only try cookies from browser in local environments
+            # Try the most common browsers in order
+            browsers = ['chrome', 'firefox', 'edge']
+            
+            for browser in browsers:
                 try:
-                    # Test if browser cookies are accessible
-                    test_opts = self.ydl_opts.copy()
-                    test_opts['cookies_from_browser'] = (browser, None, None, None)
-                    test_opts['quiet'] = True
-                    test_opts['no_warnings'] = True
-                    
-                    # Try to extract info from a simple YouTube URL to test cookies
-                    with yt_dlp.YoutubeDL(test_opts) as ydl:
-                        # Just test if we can access cookies, don't actually download
-                        logger.debug(f"Testing {browser} cookies...")
-                        
-                    # If no exception, browser cookies are accessible
-                    self.ydl_opts['cookies_from_browser'] = (browser, None, None, None)
-                    logger.info(f"Successfully configured automatic cookies from {browser}")
+                    # Simple test - just set the option, don't test it extensively
+                    self.ydl_opts['cookies_from_browser'] = (browser,)
+                    logger.info(f"Configured to use cookies from {browser}")
                     return
-                    
                 except Exception as e:
-                    logger.debug(f"Failed to configure cookies from {browser}: {e}")
+                    logger.debug(f"Cannot use {browser} cookies: {e}")
                     continue
             
-            logger.warning("No accessible browser cookies found. Proceeding without cookies.")
+            logger.info("No browser cookies configured - proceeding without cookies")
             
         except Exception as e:
-            logger.error(f"Error configuring browser cookies: {e}")
-            logger.info("Proceeding without automatic cookies")
+            logger.debug(f"Cookie configuration failed: {e}")
+            logger.info("Proceeding without cookies")
+    
+    def _configure_deployment_options(self):
+        """Configure yt-dlp options optimized for deployment environments"""
+        # Add deployment-specific options
+        self.ydl_opts.update({
+            # More conservative format selection for deployment
+            'format': 'best[height<=720]/best',
+            # Add headers to appear more like a regular browser
+            'http_headers': {
+                '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'
+            },
+            # Retry options
+            'retries': 3,
+            'fragment_retries': 3,
+            # Avoid problematic extractors in deployment
+            'extractor_retries': 2,
+        })
+        logger.info("Configured yt-dlp for deployment environment")
     
     def validate_url(self, url):
         # Handle empty or None URLs
@@ -142,22 +162,39 @@ class VideoDownloader:
                 
                 raise Exception("Could not find downloaded video file")
 
-        # Error handling for youtube download errors        
         except yt_dlp.utils.DownloadError as e:
             error_msg = str(e).lower()
             if "403" in error_msg or "forbidden" in error_msg:
                 logger.error("HTTP 403 Forbidden error detected!")
-                print("YOUTUBE ACCESS ERROR (403 Forbidden)")
-                print("This error typically occurs due to:")
-                print("1. Rate limiting or IP blocking")
-                print("2. Need for updated cookies")
-                print("3. Outdated yt-dlp version")
-                print("\n SOLUTIONS:")
-                print("1. Try again in a few minutes")
-                print("2. Update yt-dlp: pip install --upgrade yt-dlp")
-                print("3. Use a different video URL for testing")
-                print("4. Make sure your browser is logged into YouTube")
-                raise Exception("YouTube access blocked (403 Forbidden). See solutions above.")
+                
+                # Check if we're in deployment
+                is_deployment = (
+                    os.getenv('SPACE_ID') is not None or 
+                    os.getenv('RAILWAY_ENVIRONMENT') is not None or 
+                    os.getenv('RENDER') is not None or 
+                    not os.getenv('DISPLAY', '')
+                )
+                
+                if is_deployment:
+                    print("\n⚠️  YOUTUBE ACCESS ERROR (403 Forbidden) - DEPLOYMENT ENVIRONMENT")
+                    print("This is expected in deployment environments like Hugging Face Spaces because:")
+                    print("• No browser cookies are available")
+                    print("• YouTube actively blocks automated requests")
+                    print("• IP addresses from cloud providers are often rate-limited")
+                    print("\n💡 RECOMMENDED SOLUTIONS:")
+                    print("1. Use alternative video sources (direct .mp4 links)")
+                    print("2. Test with non-YouTube URLs")
+                    print("3. Consider using pre-downloaded sample videos")
+                    print("4. Some public YouTube videos may still work")
+                else:
+                    print("\n⚠️  YOUTUBE ACCESS ERROR (403 Forbidden) - LOCAL ENVIRONMENT")
+                    print("Try these solutions:")
+                    print("1. Make sure you're logged into YouTube in your browser")
+                    print("2. Try a different video URL")
+                    print("3. Wait a few minutes and try again")
+                    print("4. Update yt-dlp: pip install --upgrade yt-dlp")
+                
+                raise Exception("YouTube access blocked (403 Forbidden) - See solutions above")
             else:
                 logger.error(f"yt-dlp download error: {e}")
                 raise