From ed51bb95ee4450cab919fa3c4ccf3958f69d7792 Mon Sep 17 00:00:00 2001
From: Claude <claude@anthropic.com>
Date: Sat, 12 Jul 2025 10:15:37 +0530
Subject: [PATCH] Fix database configuration - reliable SQLite default with
 PostgreSQL option
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

**Fixed Runtime Connection Issues:**
- Remove connection testing during settings.py loading
- Default to SQLite for reliable local development
- Eliminate "Connection refused" errors on startup

**Smart Configuration Approach:**
- Default: SQLite (works out of the box)
- Railway: PostgreSQL (via DATABASE_URL environment variable)
- Local PostgreSQL: Optional via USE_POSTGRESQL=True or DATABASE_URL

**User Experience:**
- `python manage.py runserver` always works locally
- No PostgreSQL setup required for development
- Optional PostgreSQL for production parity testing
- Clear .env documentation with multiple options

**Configuration Options:**
1. Default SQLite: No configuration needed
2. Explicit DATABASE_URL: Full control over database
3. USE_POSTGRESQL=True: Simple PostgreSQL activation
4. Railway: Automatic PostgreSQL via environment

**Fixes:**
- Eliminates connection refused errors during startup
- Prevents PostgreSQL dependency for basic development
- Maintains Railway production compatibility
- Provides clear upgrade path to PostgreSQL when needed

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 netcop_hub/settings.py | 48 ++++++++++++++----------------------------
 1 file changed, 16 insertions(+), 32 deletions(-)

diff --git a/netcop_hub/settings.py b/netcop_hub/settings.py
index 57f85ee..d227be2 100644
--- a/netcop_hub/settings.py
+++ b/netcop_hub/settings.py
@@ -110,23 +110,20 @@ elif config('RAILWAY_ENVIRONMENT', default=''):
         }
     }
 else:
-    # Local development: Try PostgreSQL first, fallback to SQLite
-    try:
-        import psycopg2
-        # Test if PostgreSQL is actually available
+    # Local development: Default to SQLite for reliability
+    # Users can override with DATABASE_URL if they want PostgreSQL
+    DATABASES = {
+        'default': {
+            'ENGINE': 'django.db.backends.sqlite3',
+            'NAME': BASE_DIR / 'db.sqlite3',
+        }
+    }
+    
+    # Optional: Check if user wants PostgreSQL (via environment or file)
+    postgres_preference = config('USE_POSTGRESQL', default='False').lower()
+    if postgres_preference in ['true', '1', 'yes']:
         try:
-            # Quick connection test
-            test_conn = psycopg2.connect(
-                host='localhost',
-                database='netcop_hub',
-                user='netcop_user',
-                password='netcop_pass',
-                port='5432',
-                connect_timeout=3
-            )
-            test_conn.close()
-            
-            # PostgreSQL works - use it
+            import psycopg2
             DATABASES = {
                 'default': {
                     'ENGINE': 'django.db.backends.postgresql',
@@ -137,22 +134,9 @@ else:
                     'PORT': '5432',
                 }
             }
-        except (psycopg2.OperationalError, psycopg2.Error):
-            # PostgreSQL not available - fallback to SQLite
-            DATABASES = {
-                'default': {
-                    'ENGINE': 'django.db.backends.sqlite3',
-                    'NAME': BASE_DIR / 'db.sqlite3',
-                }
-            }
-    except ImportError:
-        # psycopg2 not available - use SQLite
-        DATABASES = {
-            'default': {
-                'ENGINE': 'django.db.backends.sqlite3',
-                'NAME': BASE_DIR / 'db.sqlite3',
-            }
-        }
+        except ImportError:
+            # psycopg2 not available - stick with SQLite
+            pass
 
 
 # Password validation