cdk-factory 0.15.13__tar.gz → 0.15.15__tar.gz

{cdk_factory-0.15.13 → cdk_factory-0.15.15}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cdk_factory
-Version: 0.15.13
+Version: 0.15.15
 Summary: CDK Factory. A QuickStarter and best practices setup for CDK projects
 Author-email: Eric Wilson <eric.wilson@geekcafe.com>
 License: MIT License

{cdk_factory-0.15.13 → cdk_factory-0.15.15}/pyproject.toml

@@ -33,7 +33,7 @@ markers = [
 [project]
 
 name = "cdk_factory"
-version = "0.15.13"
+version = "0.15.15"
 authors = [
   { name="Eric Wilson", email="eric.wilson@geekcafe.com" }
 ]

{cdk_factory-0.15.13 → cdk_factory-0.15.15}/src/cdk_factory/configurations/resources/rds.py

@@ -131,11 +131,24 @@ class RdsConfig(EnhancedBaseConfig):
 
     @property
     def secret_name(self) -> str:
-        """Name of the secret to store credentials"""
-        env_name = self.__deployment.environment if self.__deployment else None
+        """Name of the secret to store credentials (includes workload to prevent collisions)"""
+        if "secret_name" in self.__config:
+            return self.__config["secret_name"]
+        
+        # Build a unique secret name using environment and workload
+        if not self.__deployment:
+            raise ValueError("No deployment context found for RDS secret name")
+        
+        env_name = self.__deployment.environment
+        workload_name = self.__deployment.workload_name
+        
         if not env_name:
-            raise ValueError("No environment found for RDS secret name.  Please add an environment to the deployment.")
-        return self.__config.get("secret_name", f"/{env_name}/db/creds")
+            raise ValueError("No environment found for RDS secret name. Please add an environment to the deployment.")
+        if not workload_name:
+            raise ValueError("No workload name found for RDS secret name. Please add a workload name to the deployment.")
+        
+        # Default pattern: /{environment}/{workload}/rds/credentials
+        return f"/{env_name}/{workload_name}/rds/credentials"
 
     @property
     def allocated_storage(self) -> int:
@@ -180,8 +193,37 @@ class RdsConfig(EnhancedBaseConfig):
 
     @property
     def cloudwatch_logs_exports(self) -> List[str]:
-        """Log types to export to CloudWatch"""
-        return self.__config.get("cloudwatch_logs_exports", ["postgresql"])
+        """
+        Log types to export to CloudWatch (engine-specific).
+        Returns configured log types or engine-specific defaults.
+        """
+        # If explicitly configured, use that
+        if "cloudwatch_logs_exports" in self.__config:
+            return self.__config["cloudwatch_logs_exports"]
+        
+        # Otherwise, return engine-specific defaults
+        engine = self.engine.lower()
+        
+        # MySQL / MariaDB
+        if engine in ("mysql", "mariadb", "aurora-mysql"):
+            return ["error", "general", "slowquery"]
+        
+        # PostgreSQL
+        elif engine in ("postgres", "postgresql", "aurora-postgres", "aurora-postgresql"):
+            return ["postgresql"]
+        
+        # SQL Server
+        elif engine in ("sqlserver", "sqlserver-ee", "sqlserver-se", "sqlserver-ex", "sqlserver-web"):
+            return ["error", "agent"]
+        
+        # Oracle
+        elif engine in ("oracle", "oracle-ee", "oracle-se2", "oracle-se1"):
+            return ["alert", "audit", "trace"]
+        
+        # Default to empty list for unknown engines (safer than guessing)
+        else:
+            logger.warning(f"Unknown engine '{engine}', disabling CloudWatch logs exports by default")
+            return []
 
     @property
     def removal_policy(self) -> str:

cdk_factory-0.15.15/src/cdk_factory/version.py

@@ -0,0 +1 @@
+__version__ = "0.15.15"

cdk_factory-0.15.13/src/cdk_factory/version.py

@@ -1 +0,0 @@
-__version__ = "0.15.13"