refactor(logging): enhance error handling and resource management in rate limiting and sync operations

services/beteiligte_sync_utils.py

@@ -92,7 +92,7 @@ class BeteiligteSync:
             return True
             
         except Exception as e:
-            self._log(f"Fehler beim Acquire Lock: {e}", level='error')
+            self.logger.error(f"Fehler beim Acquire Lock: {e}")
             # Clean up Redis lock on error
             if self.redis:
                 try:

steps/advoware_cal_sync/calendar_sync_event_step.py

@@ -65,6 +65,7 @@ async def enforce_global_rate_limit(context=None):
         socket_timeout=int(os.getenv('REDIS_TIMEOUT_SECONDS', '5'))
     )
 
+    try:
         lua_script = """
     local key = KEYS[1]
     local current_time_ms = tonumber(ARGV[1])
@@ -97,7 +98,6 @@ async def enforce_global_rate_limit(context=None):
     end
     """
 
-    try:
         script = redis_client.register_script(lua_script)
         
         while True:
@@ -121,6 +121,12 @@ async def enforce_global_rate_limit(context=None):
 
     except Exception as e:
         log_operation('error', f"Rate limiting failed: {e}. Proceeding without limit.", context=context)
+    finally:
+        # Always close Redis connection to prevent resource leaks
+        try:
+            redis_client.close()
+        except Exception:
+            pass
 
 
 @backoff.on_exception(backoff.expo, HttpError, max_tries=4, base=3, 
@@ -958,6 +964,7 @@ async def handler(input_data: Dict[str, Any], ctx: FlowContext) -> None:
     log_operation('info', f"Starting calendar sync for employee {kuerzel}", context=ctx)
 
     redis_client = get_redis_client(ctx)
+    service = None
 
     try:
         log_operation('debug', "Initializing Advoware service", context=ctx)
@@ -1049,6 +1056,19 @@ async def handler(input_data: Dict[str, Any], ctx: FlowContext) -> None:
         
         return {'status': 200, 'body': {'status': 'completed', 'kuerzel': kuerzel}}
     
+    finally:
+        # Always close resources to prevent memory leaks
+        if service is not None:
+            try:
+                service.close()
+            except Exception as e:
+                log_operation('debug', f"Error closing Google service: {e}", context=ctx)
+        
+        try:
+            redis_client.close()
+        except Exception as e:
+            log_operation('debug', f"Error closing Redis client: {e}", context=ctx)
+
     except Exception as e:
         log_operation('error', f"Sync failed for {kuerzel}: {e}", context=ctx)
         log_operation('info', f"Handler duration (failed): {time.time() - start_time}", context=ctx)