trovesuite 1.0.28__tar.gz → 1.0.29__tar.gz

{trovesuite-1.0.28/src/trovesuite.egg-info → trovesuite-1.0.29}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: trovesuite
-Version: 1.0.28
+Version: 1.0.29
 Summary: TroveSuite services package providing authentication, authorization, notifications, Azure Storage, and other enterprise services for TroveSuite applications
 Home-page: https://dev.azure.com/brightgclt/trovesuite/_git/packages
 Author: Bright Debrah Owusu

{trovesuite-1.0.28 → trovesuite-1.0.29}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [tool.poetry]
 name = "trovesuite"
-version = "1.0.28"
+version = "1.0.29"
 description = "TroveSuite services package providing authentication, authorization, notifications, Azure Storage, and other enterprise services for TroveSuite applications"
 authors = ["brightgclt <brightgclt@gmail.com>"]
 license = "MIT"
@@ -58,7 +58,7 @@ Documentation = "https://dev.azure.com/brightgclt/trovesuite/_git/packages"
 
 [project]
 name = "trovesuite"
-version = "1.0.28"
+version = "1.0.29"
 description = "TroveSuite services package providing authentication, authorization, notifications, Azure Storage, and other enterprise services for TroveSuite applications"
 readme = "README.md"
 license = {text = "MIT"}

{trovesuite-1.0.28 → trovesuite-1.0.29}/setup.py

@@ -15,7 +15,7 @@ with open("pyproject.toml", "r", encoding="utf-8") as fh:
 
 setup(
     name="trovesuite",
-    version="1.0.28",
+    version="1.0.29",
     author="Bright Debrah Owusu",
     author_email="owusu.debrah@deladetech.com",
     description="TroveSuite services package providing authentication, authorization, notifications, and other enterprise services for TroveSuite applications",

{trovesuite-1.0.28 → trovesuite-1.0.29}/src/trovesuite/__init__.py

@@ -11,7 +11,7 @@ from .notification import NotificationService
 from .storage import StorageService
 from .utils import Helper
 
-__version__ = "1.0.20"
+__version__ = "1.0.29"
 __author__ = "Bright Debrah Owusu"
 __email__ = "owusu.debrah@deladetech.com"
 

{trovesuite-1.0.28 → trovesuite-1.0.29}/src/trovesuite/configs/database.py

@@ -6,6 +6,7 @@ from typing import Generator, Optional
 import psycopg2
 import psycopg2.pool
 from psycopg2.extras import RealDictCursor
+import threading
 from .settings import db_settings
 from .logging import get_logger
 
@@ -13,6 +14,7 @@ logger = get_logger("database")
 
 # Database connection pool
 _connection_pool: Optional[psycopg2.pool.ThreadedConnectionPool] = None
+_initialization_lock = threading.Lock()
 
 
 class DatabaseConfig:
@@ -91,6 +93,15 @@ db_config = DatabaseConfig()
 def initialize_database():
     """Initialize database connections and pool"""
     global _connection_pool
+    
+    # Close existing pool if it exists (cleanup before reinitializing)
+    if _connection_pool is not None:
+        try:
+            _connection_pool.closeall()
+            logger.info("Closed existing connection pool before reinitialization")
+        except Exception as e:
+            logger.warning(f"Error closing existing pool: {str(e)}")
+        _connection_pool = None
 
     try:
         # Test connection first
@@ -99,39 +110,131 @@ def initialize_database():
 
         # Create connection pool
         _connection_pool = db_config.create_connection_pool()
+        
+        # Verify pool was created successfully
+        if _connection_pool is None:
+            raise Exception("Connection pool creation returned None")
 
-        logger.info("Database initialization completed successfully")
+        logger.info("✅ Database initialization completed successfully")
 
     except Exception as e:
-        logger.error(f"Database initialization failed: {str(e)}")
+        logger.error(f"❌ Database initialization failed: {str(e)}")
+        _connection_pool = None  # Ensure pool is None on failure
         raise
 
 
+def _is_pool_valid(pool) -> bool:
+    """Check if the connection pool is valid and usable"""
+    if pool is None:
+        return False
+    try:
+        # ThreadedConnectionPool doesn't expose a direct "closed" attribute
+        # Check if pool has the necessary internal structures
+        if not hasattr(pool, '_pool'):
+            return False
+        if pool._pool is None:
+            return False
+        # Additional check: verify pool has connection parameters
+        if not hasattr(pool, '_kwargs'):
+            return False
+        return True
+    except (AttributeError, Exception) as e:
+        logger.debug(f"Pool validation check: {str(e)}")
+        return False
+
+
+def _recover_connection_pool() -> bool:
+    """Attempt to recover the connection pool with retry logic"""
+    global _connection_pool, _initialization_lock
+    import time
+    
+    with _initialization_lock:
+        # Double-check after acquiring lock
+        if _connection_pool is not None and _is_pool_valid(_connection_pool):
+            return True
+        
+        # Close invalid pool if it exists
+        if _connection_pool is not None:
+            try:
+                _connection_pool.closeall()
+                logger.info("Closed invalid connection pool")
+            except Exception as e:
+                logger.warning(f"Error closing invalid pool: {str(e)}")
+            _connection_pool = None
+        
+        # Retry with exponential backoff
+        max_retries = 3
+        base_delay = 1  # Start with 1 second
+        
+        for attempt in range(1, max_retries + 1):
+            try:
+                logger.warning(f"Attempting to reinitialize connection pool (attempt {attempt}/{max_retries})...")
+                initialize_database()
+                
+                if _connection_pool is not None and _is_pool_valid(_connection_pool):
+                    logger.info(f"✅ Connection pool reinitialized successfully (attempt {attempt})")
+                    return True
+                else:
+                    logger.warning(f"Pool initialized but validation failed (attempt {attempt})")
+                    
+            except Exception as e:
+                logger.error(f"Pool reinitialization attempt {attempt} failed: {str(e)}")
+                if attempt < max_retries:
+                    delay = base_delay * (2 ** (attempt - 1))  # Exponential backoff: 1s, 2s, 4s
+                    logger.info(f"Retrying in {delay} seconds...")
+                    time.sleep(delay)
+        
+        logger.error("❌ Failed to reinitialize connection pool after all retries")
+        return False
+
+
 def get_connection_pool() -> psycopg2.pool.ThreadedConnectionPool:
-    """Get the database connection pool"""
+    """Get the database connection pool, with automatic reinitialization if needed"""
     global _connection_pool
-    if _connection_pool is None:
+    
+    # Fast path: pool exists and is valid
+    if _connection_pool is not None and _is_pool_valid(_connection_pool):
+        return _connection_pool
+    
+    # Pool is None or invalid, attempt recovery
+    if not _recover_connection_pool():
         error_msg = (
-            "Database not initialized. This usually means:\n"
-            "1. Missing or incorrect .env file in app/ directory\n"
-            "2. Database credentials are wrong\n"
-            "3. Database container is not running\n"
-            "4. Database initialization failed during startup\n"
-            "Please check the startup logs for more details."
+            "Database connection pool is unavailable. This usually means:\n"
+            "1. Database server is unreachable or down\n"
+            "2. Network connectivity issues\n"
+            "3. Database credentials are incorrect\n"
+            "4. Connection pool exhausted or closed\n"
+            "5. Database initialization failed\n"
+            "Please check the startup logs and database status."
         )
         logger.error(error_msg)
         raise Exception(error_msg)
+    
+    if _connection_pool is None:
+        error_msg = "Connection pool recovery completed but pool is still None"
+        logger.error(error_msg)
+        raise Exception(error_msg)
+    
     return _connection_pool
 
 
 def _validate_connection(conn) -> bool:
     """Validate if a connection is still alive"""
     try:
+        # Check if connection is closed first
+        if conn.closed:
+            return False
+        
         # Test if connection is alive with a simple query
         with conn.cursor() as cursor:
             cursor.execute("SELECT 1")
+            cursor.fetchone()
         return True
-    except (psycopg2.OperationalError, psycopg2.InterfaceError):
+    except (psycopg2.OperationalError, psycopg2.InterfaceError, psycopg2.DatabaseError) as e:
+        logger.warning(f"Connection validation failed: {str(e)}")
+        return False
+    except Exception as e:
+        logger.warning(f"Unexpected error during connection validation: {str(e)}")
         return False
 
 

{trovesuite-1.0.28 → trovesuite-1.0.29}/src/trovesuite/configs/settings.py

@@ -10,11 +10,6 @@ class Settings:
     DB_PORT: str = os.getenv("DB_PORT")
     DB_PASSWORD: str = os.getenv("DB_PASSWORD")
     
-    # CORS Endpoint
-    # LOCAL_HOST_URL = os.getenv("LOCAL_HOST_URL")
-    # FRONTEND_SERVER_URL_1 = os.getenv("FRONTEND_SERVER_URL")
-    # FRONTEND_SERVER_URL_2 = os.getenv("FRONTEND_SERVER_URL")
-
     # Application settings
     DEBUG: bool = os.getenv("DEBUG", "True").lower() in ("true",1)
     APP_NAME: str = os.getenv("APP_NAME", "Python Template API")

{trovesuite-1.0.28 → trovesuite-1.0.29/src/trovesuite.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: trovesuite
-Version: 1.0.28
+Version: 1.0.29
 Summary: TroveSuite services package providing authentication, authorization, notifications, Azure Storage, and other enterprise services for TroveSuite applications
 Home-page: https://dev.azure.com/brightgclt/trovesuite/_git/packages
 Author: Bright Debrah Owusu