Update config_provider.py

config_provider.py

@@ -29,7 +29,14 @@ class ConfigProvider:
     _lock = threading.RLock()  # Reentrant lock for nested calls
     _file_lock = threading.Lock()  # Separate lock for file operations
     _CONFIG_PATH = Path(__file__).parent / "service_config.jsonc"
-    
+
+    @staticmethod
+    def _normalize_date(date_str: Optional[str]) -> str:
+        """Normalize date string for comparison"""
+        if not date_str:
+            return ""
+        return date_str.replace(' ', 'T').replace('+00:00', 'Z').replace('.000Z', 'Z')
+
     @classmethod
     def get(cls) -> ServiceConfig:
         """Get cached configuration - thread-safe"""
@@ -147,7 +154,11 @@ class ConfigProvider:
                 
                 # Check for race condition
                 if config.last_update_date and current_config.last_update_date:
-                    if config.last_update_date != current_config.last_update_date:
+                    # Tarihleri normalize et                   
+                    expected_normalized = cls._normalize_date(config.last_update_date)
+                    actual_normalized = cls.normalize_date(current_config.last_update_date)
+                    
+                    if expected_normalized != actual_normalized:
                         raise RaceConditionError(
                             "Configuration was modified by another user",
                             current_user=username,
@@ -277,25 +288,27 @@ class ConfigProvider:
             
             if not project:
                 raise ResourceNotFoundError("project", project_id)
-                
-            log_debug(f"🔍 Project update check - Project ID: {project_id}")
-            log_debug(f"🔍 Expected last_update: {expected_last_update}")
-            log_debug(f"🔍 Actual last_update: {project.last_update_date}")
-            log_debug(f"🔍 Type of expected: {type(expected_last_update)}")
-            log_debug(f"🔍 Type of actual: {type(project.last_update_date)}")
             
-            # Check race condition - Eğer expected_last_update None ise kontrol etme
+            # Check race condition
             if expected_last_update is not None and expected_last_update != '':
-                # Eğer project'in last_update_date'i yoksa (ilk kez update ediliyor), race condition olmaz
-                if project.last_update_date and project.last_update_date != expected_last_update:
-                    raise RaceConditionError(
-                        f"Project '{project.name}' was modified by another user",
-                        current_user=username,
-                        last_update_user=project.last_update_user,
-                        last_update_date=project.last_update_date,
-                        entity_type="project",
-                        entity_id=project_id
-                    )
+                if project.last_update_date:
+                    # Datetime objesini string'e çevir
+                    actual_date_str = project.last_update_date.isoformat() if isinstance(project.last_update_date, datetime) else str(project.last_update_date)
+                    
+                    # Tarihleri normalize et
+                    expected_normalized = cls._normalize_date(expected_last_update)
+                    actual_normalized = cls._normalize_date(actual_date_str)
+                    
+                    if expected_normalized != actual_normalized:
+                        log_info(f"🔍 Date mismatch - Expected: {expected_normalized}, Actual: {actual_normalized}")
+                        raise RaceConditionError(
+                            f"Project '{project.name}' was modified by another user",
+                            current_user=username,
+                            last_update_user=project.last_update_user,
+                            last_update_date=project.last_update_date,
+                            entity_type="project",
+                            entity_id=project_id
+                        )
             
             # Update fields
             for key, value in update_data.items():
@@ -462,16 +475,21 @@ class ConfigProvider:
                 raise ValidationError("Published versions cannot be modified")
             
             # Check race condition
-            if expected_last_update is not None:
-                if version.last_update_date != expected_last_update:
-                    raise RaceConditionError(
-                        f"Version {version_no} was modified by another user",
-                        current_user=username,
-                        last_update_user=version.last_update_user,
-                        last_update_date=version.last_update_date,
-                        entity_type="version",
-                        entity_id=f"{project_id}:{version_no}"
-                    )
+            if expected_last_update is not None and expected_last_update != '':
+                if version.last_update_date:
+                    # Tarihleri normalize et
+                    expected_normalized = cls._normalize_date(expected_last_update)
+                    actual_normalized = cls._normalize_date(version.last_update_date)
+                    
+                    if expected_normalized != actual_normalized:
+                        raise RaceConditionError(
+                            f"Version {version_no} was modified by another user",
+                            current_user=username,
+                            last_update_user=version.last_update_user,
+                            last_update_date=version.last_update_date,
+                            entity_type="version",
+                            entity_id=f"{project_id}:{version_no}"
+                        )
             
             # Update fields
             for key, value in update_data.items():
@@ -556,17 +574,22 @@ class ConfigProvider:
                 raise ResourceNotFoundError("api", api_name)
             
             # Check race condition
-            if expected_last_update is not None:
-                if api.last_update_date != expected_last_update:
-                    raise RaceConditionError(
-                        f"API '{api.name}' was modified by another user",
-                        current_user=username,
-                        last_update_user=api.last_update_user,
-                        last_update_date=api.last_update_date,
-                        entity_type="api",
-                        entity_id=api.name
-                    )
-            
+            if expected_last_update is not None and expected_last_update != '':
+                if api.last_update_date:
+                    # Tarihleri normalize et
+                    expected_normalized = cls._normalize_date(expected_last_update)
+                    actual_normalized = cls._normalize_date(api.last_update_date)
+                    
+                    if expected_normalized != actual_normalized:
+                        raise RaceConditionError(
+                            f"API '{api.name}' was modified by another user",
+                            current_user=username,
+                            last_update_user=api.last_update_user,
+                            last_update_date=api.last_update_date,
+                            entity_type="api",
+                            entity_id=api.name
+                        )
+                        
             # Update fields
             for key, value in update_data.items():
                 if hasattr(api, key) and key not in ['name', 'created_date', 'created_by', 'last_update_date']: