Simplify database configuration - fallback to SQLite

Author: rprabhat

backend/app/database.py

@@ -11,18 +11,22 @@
 )
 sys.stderr.write(f"[DB DEBUG] DATA_DIR: {os.environ.get('DATA_DIR', 'Not set')}\n")
 
-RAILWAY_TEST_VARIABLE = "THIS_SHOULD_APPEAR_IN_LOGS_IF_CODE_IS_LOADED"
-
 if DATABASE_URL:
     if DATABASE_URL.startswith("postgresql://"):
-        sys.stderr.write("[DB DEBUG] Using PostgreSQL with NullPool and SSL\n")
+        sys.stderr.write("[DB DEBUG] Using PostgreSQL\n")
+
+        from urllib.parse import urlparse
+
+        parsed = urlparse(DATABASE_URL)
+
         engine = create_engine(
             DATABASE_URL,
             poolclass=NullPool,
             connect_args={
-                "connect_timeout": 10,
+                "connect_timeout": 30,
                 "sslmode": "require",
-                "application_name": "proproo-backend",
+                "keepalives": 1,
+                "keepalives_idle": 60,
             },
         )
     else:

backend/app/main.py

@@ -21,9 +21,7 @@
 
 @app.on_event("startup")
 async def startup_event():
-    """Initialize database on startup"""
     try:
-        # Create tables if they don't exist
         Base.metadata.create_all(bind=engine)
         print("[STARTUP] Database tables created/verified")
     except Exception as e:
@@ -33,81 +31,3 @@ async def startup_event():
 @app.get("/health")
 def health_check():
     return {"status": "ok"}
-
-
-@app.get("/test-db")
-def test_db():
-    """Test database connection"""
-    import os
-    import ssl
-    import socket
-    import subprocess
-
-    DATABASE_URL = os.environ.get("DATABASE_URL", "")
-
-    results = {"tests": []}
-
-    try:
-        from urllib.parse import urlparse
-
-        parsed = urlparse(DATABASE_URL)
-        hostname = parsed.hostname
-        port = parsed.port or 5432
-
-        results["tests"].append({"host": hostname, "port": port})
-
-        try:
-            proc = subprocess.run(
-                ["psql", "--version"], capture_output=True, text=True, timeout=5
-            )
-            results["tests"].append({"psql": proc.stdout.strip()})
-        except Exception as e:
-            results["tests"].append({"psql_error": str(e)[:100]})
-
-        try:
-            result = subprocess.run(
-                [
-                    "psql",
-                    f"postgresql://{parsed.username}:{parsed.password}@{hostname}:{port}{parsed.path}",
-                    "-c",
-                    "SELECT 1;",
-                ],
-                capture_output=True,
-                text=True,
-                timeout=30,
-                env={**os.environ, "PGSSLMODE": "require"},
-            )
-            results["tests"].append(
-                {"psql_require": result.stdout[:200] + result.stderr[:200]}
-            )
-        except Exception as e:
-            results["tests"].append({"psql_require_error": str(e)[:200]})
-
-        try:
-            import psycopg2
-
-            for mode in ["require"]:
-                try:
-                    conn = psycopg2.connect(
-                        host=hostname,
-                        port=port,
-                        dbname=parsed.path[1:],
-                        user=parsed.username,
-                        password=parsed.password,
-                        sslmode=mode,
-                        sslcompression=0,
-                    )
-                    cur = conn.cursor()
-                    cur.execute("SELECT 1;")
-                    cur.close()
-                    conn.close()
-                    results["tests"].append({f"psycopg2_{mode}_nocomp": "success"})
-                except Exception as e:
-                    results["tests"].append({f"psycopg2_{mode}_nocomp": str(e)[:200]})
-        except ImportError:
-            results["tests"].append({"psycopg2": "not installed"})
-
-    except Exception as e:
-        results["error"] = str(e)
-
-    return results