Skip to content

Commit 501fc49

Browse files
committed
Revert SSL certificate changes - not needed
Redis-py accepts string values for ssl_cert_reqs ('required', 'optional', 'none'). The original SSL certificate error was likely a temporary issue or resolved by other changes. The original connection code works fine.
1 parent 626757c commit 501fc49

File tree

1 file changed

+2
-16
lines changed

1 file changed

+2
-16
lines changed

src/common/connection.py

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import sys
2-
import ssl
32
from src.version import __version__
43
import redis
54
from redis import Redis
@@ -8,19 +7,6 @@
87
from src.common.config import REDIS_CFG
98

109

11-
def _get_ssl_cert_reqs(cert_reqs_str: str):
12-
"""Convert string SSL certificate requirements to SSL constants."""
13-
if cert_reqs_str == 'required':
14-
return ssl.CERT_REQUIRED
15-
elif cert_reqs_str == 'optional':
16-
return ssl.CERT_OPTIONAL
17-
elif cert_reqs_str == 'none':
18-
return ssl.CERT_NONE
19-
else:
20-
# Default to required for safety
21-
return ssl.CERT_REQUIRED
22-
23-
2410
class RedisConnectionManager:
2511
_instance: Optional[Redis] = None
2612

@@ -39,7 +25,7 @@ def get_connection(cls, decode_responses=True) -> Redis:
3925
"ssl_ca_path": REDIS_CFG["ssl_ca_path"],
4026
"ssl_keyfile": REDIS_CFG["ssl_keyfile"],
4127
"ssl_certfile": REDIS_CFG["ssl_certfile"],
42-
"ssl_cert_reqs": _get_ssl_cert_reqs(REDIS_CFG["ssl_cert_reqs"]),
28+
"ssl_cert_reqs": REDIS_CFG["ssl_cert_reqs"],
4329
"ssl_ca_certs": REDIS_CFG["ssl_ca_certs"],
4430
"decode_responses": decode_responses,
4531
"lib_name": f"redis-py(mcp-server_v{__version__})",
@@ -58,7 +44,7 @@ def get_connection(cls, decode_responses=True) -> Redis:
5844
"ssl_ca_path": REDIS_CFG["ssl_ca_path"],
5945
"ssl_keyfile": REDIS_CFG["ssl_keyfile"],
6046
"ssl_certfile": REDIS_CFG["ssl_certfile"],
61-
"ssl_cert_reqs": _get_ssl_cert_reqs(REDIS_CFG["ssl_cert_reqs"]),
47+
"ssl_cert_reqs": REDIS_CFG["ssl_cert_reqs"],
6248
"ssl_ca_certs": REDIS_CFG["ssl_ca_certs"],
6349
"decode_responses": decode_responses,
6450
"lib_name": f"redis-py(mcp-server_v{__version__})",

0 commit comments

Comments
 (0)