From 704c6c8aa828ba4d7a46d14934330b931b8fef55 Mon Sep 17 00:00:00 2001 From: Shuchang Zheng Date: Thu, 19 Feb 2026 09:02:52 -0800 Subject: [PATCH] Use pooled Redis client for notification registry in cloud (#4808) --- skyvern/forge/forge_app.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/skyvern/forge/forge_app.py b/skyvern/forge/forge_app.py index dcfe9c99..f595f176 100644 --- a/skyvern/forge/forge_app.py +++ b/skyvern/forge/forge_app.py @@ -111,15 +111,14 @@ def create_forge_app() -> ForgeApp: app.STORAGE = StorageFactory.get_storage() app.CACHE = CacheFactory.get_cache() - if settings.NOTIFICATION_REGISTRY_TYPE == "redis": + if settings.NOTIFICATION_REGISTRY_TYPE == "redis" and settings.NOTIFICATION_REDIS_URL: from redis.asyncio import from_url as redis_from_url from skyvern.forge.sdk.notification.factory import NotificationRegistryFactory from skyvern.forge.sdk.notification.redis import RedisNotificationRegistry from skyvern.forge.sdk.redis.factory import RedisClientFactory - redis_url = settings.NOTIFICATION_REDIS_URL or settings.REDIS_URL - redis_client = redis_from_url(redis_url, decode_responses=True) + redis_client = redis_from_url(settings.NOTIFICATION_REDIS_URL, decode_responses=True) RedisClientFactory.set_client(redis_client) NotificationRegistryFactory.set_registry(RedisNotificationRegistry(redis_client)) app.ARTIFACT_MANAGER = ArtifactManager()