From aa4a9877f5dc338d2cac64850e64873863bcb169 Mon Sep 17 00:00:00 2001 From: wangxiaolei Date: Wed, 18 Mar 2026 10:08:16 +0800 Subject: [PATCH] fix: fix max_retries is hardcode (#33619) --- api/configs/middleware/cache/redis_config.py | 12 +++++++++++- api/dify_graph/nodes/http_request/node.py | 1 - 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/api/configs/middleware/cache/redis_config.py b/api/configs/middleware/cache/redis_config.py index 367cb52731..3b91207545 100644 --- a/api/configs/middleware/cache/redis_config.py +++ b/api/configs/middleware/cache/redis_config.py @@ -1,4 +1,4 @@ -from pydantic import Field, NonNegativeInt, PositiveFloat, PositiveInt +from pydantic import Field, NonNegativeInt, PositiveFloat, PositiveInt, field_validator from pydantic_settings import BaseSettings @@ -116,3 +116,13 @@ class RedisConfig(BaseSettings): description="Maximum connections in the Redis connection pool (unset for library default)", default=None, ) + + @field_validator("REDIS_MAX_CONNECTIONS", mode="before") + @classmethod + def _empty_string_to_none_for_max_conns(cls, v): + """Allow empty string in env/.env to mean 'unset' (None).""" + if v is None: + return None + if isinstance(v, str) and v.strip() == "": + return None + return v diff --git a/api/dify_graph/nodes/http_request/node.py b/api/dify_graph/nodes/http_request/node.py index b17c820a80..486ae241ee 100644 --- a/api/dify_graph/nodes/http_request/node.py +++ b/api/dify_graph/nodes/http_request/node.py @@ -101,7 +101,6 @@ class HttpRequestNode(Node[HttpRequestNodeData]): timeout=self._get_request_timeout(self.node_data), variable_pool=self.graph_runtime_state.variable_pool, http_request_config=self._http_request_config, - max_retries=0, ssl_verify=self.node_data.ssl_verify, http_client=self._http_client, file_manager=self._file_manager,