Update app.py

app.py

@@ -10,10 +10,7 @@ import logging
 from typing import List, Optional
 
 # Configure logging
-logging.basicConfig(
-    level=logging.INFO,
-    format='%(asctime)s - %(name)s - %(levelname)s - %(message)s'
-)
+logging.basicConfig(level=logging.INFO)
 logger = logging.getLogger(__name__)
 
 # Create FastAPI app
@@ -32,58 +29,21 @@ app.add_middleware(
     allow_headers=["*"],
 )
 
-@app.on_event("startup")
-async def startup_event():
-    """Startup event - runs after server starts"""
-    logger.info("=" * 60)
-    logger.info("HTML to PDF Converter API Starting...")
-    logger.info("=" * 60)
-    logger.info("Checking dependencies...")
-    
-    # Check if node is available
-    try:
-        result = subprocess.run(['node', '--version'], capture_output=True, text=True, timeout=5)
-        logger.info(f"Node.js version: {result.stdout.strip()}")
-    except Exception as e:
-        logger.error(f"Node.js not found: {e}")
-    
-    # Check if chromium is available
-    chromium_path = os.environ.get('PUPPETEER_EXECUTABLE_PATH', '/usr/bin/chromium')
-    if os.path.exists(chromium_path):
-        logger.info(f"Chromium found at: {chromium_path}")
-    else:
-        logger.error(f"Chromium not found at: {chromium_path}")
-    
-    # Check if puppeteer script exists
-    if os.path.exists('/app/puppeteer_pdf.js'):
-        logger.info("Puppeteer script found")
-    else:
-        logger.error("Puppeteer script not found!")
-    
-    logger.info("=" * 60)
-    logger.info("API is ready!")
-    logger.info("=" * 60)
-
 @app.get("/")
 async def root():
-    """Root endpoint - simple response for quick health check"""
+    """Root endpoint - returns immediately for health check"""
     return {
         "status": "online",
         "service": "HTML to PDF Converter API",
         "version": "1.0.0",
         "docs": "/docs",
-        "health": "/health"
+        "ui": "/ui"
     }
 
 @app.get("/health")
 async def health():
     """Health check endpoint"""
-    return {
-        "status": "healthy",
-        "service": "HTML to PDF Converter API",
-        "version": "1.0.0",
-        "endpoints": ["/convert", "/convert-html", "/convert-base64"]
-    }
+    return {"status": "healthy"}
 
 @app.get("/ui", response_class=HTMLResponse)
 async def ui():
@@ -148,14 +108,6 @@ async def ui():
             }
             ul { margin-left: 20px; }
             li { margin: 8px 0; }
-            a { color: #667eea; text-decoration: none; font-weight: 500; }
-            a:hover { text-decoration: underline; }
-            .links { 
-                display: flex; 
-                gap: 15px; 
-                margin: 20px 0;
-                flex-wrap: wrap;
-            }
             .btn {
                 display: inline-block;
                 background: #667eea;
@@ -163,6 +115,7 @@ async def ui():
                 padding: 12px 24px;
                 border-radius: 6px;
                 text-decoration: none;
+                margin: 5px;
                 transition: all 0.3s;
             }
             .btn:hover {
@@ -182,19 +135,13 @@ async def ui():
                 page breaks, image embedding, and multiple aspect ratios.
             </p>
             
-            <div class="links">
+            <div>
                 <a href="/docs" class="btn">📚 API Documentation</a>
                 <a href="/health" class="btn">💚 Health Check</a>
             </div>
             
             <h2>📡 Endpoints</h2>
             
-            <div class="endpoint">
-                <h3>GET /health</h3>
-                <p>Check API health status</p>
-                <pre>curl https://YOUR-SPACE.hf.space/health</pre>
-            </div>
-            
             <div class="endpoint">
                 <h3>POST /convert</h3>
                 <p>Upload HTML file and convert to PDF</p>
@@ -202,56 +149,26 @@ async def ui():
                 <ul>
                     <li><code>html_file</code> - HTML file (required)</li>
                     <li><code>aspect_ratio</code> - "16:9", "1:1", or "9:16" (optional)</li>
-                    <li><code>auto_detect</code> - Auto-detect aspect ratio (default: true)</li>
-                    <li><code>images</code> - Image files to embed (optional, multiple files)</li>
+                    <li><code>images</code> - Image files to embed (optional)</li>
                 </ul>
-                <pre>curl -X POST https://YOUR-SPACE.hf.space/convert \\
-  -F "html_file=@document.html" \\
-  -F "images=@logo.png" \\
-  -o output.pdf</pre>
             </div>
             
             <div class="endpoint">
                 <h3>POST /convert-html</h3>
                 <p>Convert HTML string to PDF</p>
-                <p><strong>Parameters:</strong></p>
-                <ul>
-                    <li><code>html_content</code> - HTML as string (required)</li>
-                    <li><code>aspect_ratio</code> - "16:9", "1:1", or "9:16" (optional)</li>
-                </ul>
-                <pre>curl -X POST https://YOUR-SPACE.hf.space/convert-html \\
-  -F "html_content=&lt;html&gt;&lt;body&gt;&lt;h1&gt;Hello!&lt;/h1&gt;&lt;/body&gt;&lt;/html&gt;" \\
-  -o output.pdf</pre>
             </div>
             
             <div class="endpoint">
                 <h3>POST /convert-base64</h3>
                 <p>Convert HTML to base64-encoded PDF (returns JSON)</p>
-                <p><strong>Response:</strong></p>
-                <pre>{
-  "success": true,
-  "pdf_base64": "JVBERi0xLjQK...",
-  "aspect_ratio": "9:16",
-  "size_bytes": 45678
-}</pre>
             </div>
             
             <h2>📐 Aspect Ratios</h2>
             <ul>
-                <li><code>16:9</code> - Landscape (297mm × 210mm) - Presentations</li>
-                <li><code>1:1</code> - Square (210mm × 210mm) - Social media</li>
-                <li><code>9:16</code> - Portrait (210mm × 297mm) - Documents</li>
+                <li><code>16:9</code> - Landscape - Presentations</li>
+                <li><code>1:1</code> - Square - Social media</li>
+                <li><code>9:16</code> - Portrait - Documents (default)</li>
             </ul>
-            
-            <h2>📄 HTML Structure for Page Breaks</h2>
-            <p>Use class <code>page</code> or <code>slide</code> for automatic page breaks:</p>
-            <pre>&lt;div class="page"&gt;
-  &lt;h1&gt;Page 1&lt;/h1&gt;
-&lt;/div&gt;
-
-&lt;div class="page"&gt;
-  &lt;h1&gt;Page 2&lt;/h1&gt;
-&lt;/div&gt;</pre>
         </div>
     </body>
     </html>
@@ -319,8 +236,6 @@ def convert_to_pdf(html: str, ratio: str) -> bytes:
             logger.error(f"Conversion failed: {result.stderr}")
             raise Exception(f"PDF conversion failed: {result.stderr}")
         
-        logger.info("Conversion successful")
-        
         pdf_file = html_file.replace('.html', '.pdf')
         
         if not os.path.exists(pdf_file):
@@ -329,7 +244,7 @@ def convert_to_pdf(html: str, ratio: str) -> bytes:
         with open(pdf_file, 'rb') as f:
             pdf_bytes = f.read()
         
-        logger.info(f"PDF size: {len(pdf_bytes)} bytes")
+        logger.info(f"PDF generated: {len(pdf_bytes)} bytes")
         return pdf_bytes
     
     finally:
@@ -344,13 +259,10 @@ async def convert(
 ):
     """Convert uploaded HTML file to PDF"""
     try:
-        logger.info(f"Received conversion request: {html_file.filename}")
-        
         html = (await html_file.read()).decode('utf-8')
         
         if auto_detect:
             aspect_ratio = detect_aspect_ratio(html)
-            logger.info(f"Auto-detected aspect ratio: {aspect_ratio}")
         elif not aspect_ratio:
             aspect_ratio = "9:16"
         
@@ -359,7 +271,6 @@ async def convert(
         
         # Embed images if provided
         if images:
-            logger.info(f"Embedding {len(images)} image(s)")
             for img in images:
                 img_bytes = await img.read()
                 b64 = base64.b64encode(img_bytes).decode()
@@ -370,7 +281,6 @@ async def convert(
         
         pdf = convert_to_pdf(html, aspect_ratio)
         
-        logger.info("Returning PDF response")
         return Response(
             content=pdf,
             media_type="application/pdf",
@@ -381,7 +291,7 @@ async def convert(
         )
     
     except Exception as e:
-        logger.error(f"Error in /convert: {str(e)}")
+        logger.error(f"Error: {str(e)}")
         raise HTTPException(500, str(e))
 
 @app.post("/convert-html")
@@ -392,8 +302,6 @@ async def convert_html(
 ):
     """Convert HTML string to PDF"""
     try:
-        logger.info("Received HTML string conversion request")
-        
         if auto_detect:
             aspect_ratio = detect_aspect_ratio(html_content)
         elif not aspect_ratio:
@@ -414,7 +322,7 @@ async def convert_html(
         )
     
     except Exception as e:
-        logger.error(f"Error in /convert-html: {str(e)}")
+        logger.error(f"Error: {str(e)}")
         raise HTTPException(500, str(e))
 
 @app.post("/convert-base64")
@@ -425,8 +333,6 @@ async def convert_base64(
 ):
     """Convert HTML to base64-encoded PDF"""
     try:
-        logger.info("Received base64 conversion request")
-        
         if auto_detect:
             aspect_ratio = detect_aspect_ratio(html_content)
         elif not aspect_ratio:
@@ -446,7 +352,7 @@ async def convert_base64(
         })
     
     except Exception as e:
-        logger.error(f"Error in /convert-base64: {str(e)}")
+        logger.error(f"Error: {str(e)}")
         raise HTTPException(500, str(e))
 
 if __name__ == "__main__":