Debug - database

app/database.py

@@ -10,12 +10,12 @@ load_dotenv()
 logger = logging.getLogger(__name__)
 
 # --- Database URL Configuration ---
-# --- CHANGE THIS LINE: Use path relative to WORKDIR (/code) ---
-# Using an absolute path inside /code is also fine: '/code/app.db'
-DEFAULT_DB_PATH = "/code/app.db" # Store DB in the main workdir
+# --- CHANGE THIS LINE: Use the /tmp directory ---
+DEFAULT_DB_PATH = "/tmp/app.db" # Store DB in the temporary directory
 
 raw_db_url = os.getenv("DATABASE_URL", f"sqlite+aiosqlite:///{DEFAULT_DB_PATH}")
 
+# --- (Rest of the URL parsing and async Database setup remains the same) ---
 final_database_url = raw_db_url
 if raw_db_url.startswith("sqlite+aiosqlite"):
     parsed_url = urlparse(raw_db_url)
@@ -28,7 +28,6 @@ if raw_db_url.startswith("sqlite+aiosqlite"):
 else:
     logger.info(f"Using non-SQLite async DB URL: {final_database_url}")
 
-# --- Async Database Instance ---
 database = Database(final_database_url)
 metadata = MetaData()
 users = Table(
@@ -47,25 +46,24 @@ engine = create_engine(sync_db_url)
 # --- Directory and Table Creation Logic ---
 db_file_path = ""
 if sync_db_url.startswith("sqlite"):
+    # Path should be absolute starting with /tmp/
     path_part = sync_db_url.split("sqlite:///")[-1].split("?")[0]
-    # Use os.path.abspath to resolve relative paths based on WORKDIR
-    db_file_path = os.path.abspath(path_part) # Should resolve to /code/app.db
+    db_file_path = path_part # Should be /tmp/app.db
 
 if db_file_path:
-    # --- CHANGE THIS LINE: Check writability of the target directory ---
-    db_dir = os.path.dirname(db_file_path) # Should be /code
+    # --- CHANGE THIS LINE: Check writability of the /tmp directory ---
+    db_dir = os.path.dirname(db_file_path) # Should be /tmp
     logger.info(f"Ensuring database directory exists: {db_dir}")
     try:
-        # Directory /code should exist because it's the WORKDIR
-        # We mainly need to check if it's writable
+        # /tmp should always exist, but check writability
         if not os.path.exists(db_dir):
-             logger.warning(f"Database directory {db_dir} does not exist! Attempting creation (may fail).")
-             # This shouldn't really happen for /code unless WORKDIR is wrong
-             os.makedirs(db_dir, exist_ok=True)
+             # This would be very strange, but log it.
+             logger.error(f"CRITICAL: Directory {db_dir} does not exist!")
+             # No need to create /tmp usually
 
         if not os.access(db_dir, os.W_OK):
-            # If /code isn't writable, we have a bigger problem
-            logger.error(f"Database directory {db_dir} is not writable! Database creation will likely fail.")
+            # If even /tmp isn't writable, something is very wrong with the environment
+            logger.error(f"CRITICAL: Directory {db_dir} is not writable! Cannot create database.")
         else:
             logger.info(f"Database directory {db_dir} appears writable.")
 
@@ -74,7 +72,7 @@ if db_file_path:
     except Exception as e:
         logger.error(f"Unexpected error checking directory {db_dir}: {e}")
 
-# Now try connecting and creating the table with the sync engine
+# --- (Rest of the table creation logic and async functions remain the same) ---
 try:
     logger.info("Attempting to connect with sync engine to check/create table...")
     with engine.connect() as connection:
@@ -87,11 +85,10 @@ try:
             logger.info("Users table created (or creation attempted).")
 
 except Exception as e:
-    # Hopefully, this won't happen now if /code is writable
+    # This *should* finally succeed if /tmp is writable
     logger.exception(f"CRITICAL: Failed to connect/create database tables using sync engine: {e}")
 
-
-# --- Async connect/disconnect functions ---
+# Async connect/disconnect functions
 async def connect_db():
     try:
         await database.connect()