From e76d80defe98c29bd4134b61f3df39e6db7d8d62 Mon Sep 17 00:00:00 2001 From: Harry Date: Wed, 17 Sep 2025 20:57:52 +0800 Subject: [PATCH] fix(trigger): update client parameter handling in TriggerProviderService - Modified the `create_provider_encrypter` call to include a cache assignment, ensuring proper management of encryption resources. - Added a cache deletion step after updating client parameters, enhancing the integrity of the parameter handling process. These changes improve the reliability of client parameter updates within the trigger provider service. --- api/services/trigger/trigger_provider_service.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/api/services/trigger/trigger_provider_service.py b/api/services/trigger/trigger_provider_service.py index 7cd794a636..9461387ade 100644 --- a/api/services/trigger/trigger_provider_service.py +++ b/api/services/trigger/trigger_provider_service.py @@ -400,7 +400,7 @@ class TriggerProviderService: # Update client params if provided if client_params is not None: - encrypter, _ = create_provider_encrypter( + encrypter, cache = create_provider_encrypter( tenant_id=tenant_id, config=[x.to_basic_provider_config() for x in provider_controller.get_oauth_client_schema()], cache=NoOpProviderCredentialCache(), @@ -413,6 +413,7 @@ class TriggerProviderService: for key, value in client_params.items() } custom_client.encrypted_oauth_params = json.dumps(encrypter.encrypt(new_params)) + cache.delete() # Update enabled status if provided if enabled is not None: