Skip to content

Commit 595c54d

Browse files
Renames cli flag
Signed-off-by: Elena Kolevska <elena@kolevska.com>
1 parent 7d1dbba commit 595c54d

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,7 @@ make dev-stop
301301
--db DB # Redis database number (default: 0)
302302
--cluster # Use Redis Cluster mode
303303
--cluster-nodes NODES # Comma-separated cluster nodes (host:port)
304-
--ssl # Use SSL/TLS connection
304+
--ssl-enabled # Use SSL/TLS connection
305305
--ssl-cert-reqs LEVEL # SSL certificate requirements (none/optional/required)
306306
--ssl-ca-certs PATH # Path to CA certificates file
307307
--ssl-certfile PATH # Path to client certificate file

cli.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ def describe_profile(profile_name):
118118
@click.option('--db', type=int, default=lambda: get_env_or_default('REDIS_DB', 0, int), help='Redis database number')
119119
@click.option('--cluster', is_flag=True, default=lambda: get_env_or_default('REDIS_CLUSTER', False, bool), help='Use Redis Cluster mode')
120120
@click.option('--cluster-nodes', default=lambda: get_env_or_default('REDIS_CLUSTER_NODES', None), help='Comma-separated list of cluster nodes (host:port)')
121-
@click.option('--ssl', is_flag=True, default=lambda: get_env_or_default('REDIS_SSL_ENABLED', False, bool), help='Use SSL/TLS connection')
121+
@click.option('--ssl-enabled', type=bool, default=lambda: get_env_or_default('REDIS_SSL_ENABLED', False, bool), help='Use SSL/TLS connection')
122122
@click.option('--ssl-keyfile', default=lambda: get_env_or_default('REDIS_SSL_KEYFILE', None), help='Path to client private key file')
123123
@click.option('--ssl-certfile', default=lambda: get_env_or_default('REDIS_SSL_CERTFILE', None), help='Path to client certificate file')
124124
@click.option('--ssl-cert-reqs', default=lambda: get_env_or_default('REDIS_SSL_CERT_REQS', 'required'), type=click.Choice(['none', 'optional', 'required']), help='SSL certificate requirements')
@@ -291,7 +291,7 @@ def _build_config_from_args(kwargs) -> RunnerConfig:
291291
database=kwargs['db'],
292292
cluster_mode=kwargs['cluster'],
293293
cluster_nodes=cluster_nodes,
294-
ssl=kwargs['ssl'],
294+
ssl=kwargs['ssl_enabled'],
295295
ssl_keyfile=kwargs['ssl_keyfile'],
296296
ssl_certfile=kwargs['ssl_certfile'],
297297
ssl_cert_reqs=kwargs['ssl_cert_reqs'],

redis_client.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,8 +115,6 @@ def _build_pool_kwargs(self) -> Dict[str, Any]:
115115
if self.config.ssl:
116116
ssl_kwargs = {'ssl': True}
117117

118-
119-
120118
# Add SSL parameters directly as redis-py expects them
121119
if self.config.ssl_keyfile is not None:
122120
ssl_kwargs['ssl_keyfile'] = self.config.ssl_keyfile

0 commit comments

Comments
 (0)