Skip to content

Commit 3d1258b

Browse files
author
klapaudius
committed
Add default values for Redis adapter configuration
Introduced fallback defaults for Redis adapter parameters (prefix, connection, and ttl) to ensure application stability when configuration values are not provided. Adjusted tests to reflect the updated behavior and validate the new defaults.
1 parent 7cdb4f7 commit 3d1258b

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

src/DependencyInjection/KlpMcpServerExtension.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@ public function load(array $configs, ContainerBuilder $container): void
2222
$container->setParameter('klp_mcp_server.middlewares', $config['middlewares']);
2323
$container->setParameter('klp_mcp_server.provider', 'klp_mcp_server.provider.'.$config['server_provider']);
2424
$container->setParameter('klp_mcp_server.adapter', 'klp_mcp_server.adapter.'.$config['sse_adapter']);
25-
$container->setParameter('klp_mcp_server.adapters.redis.prefix', $config['adapters']['redis']['prefix']);
26-
$container->setParameter('klp_mcp_server.adapters.redis.connection', $config['adapters']['redis']['connection']);
27-
$container->setParameter('klp_mcp_server.adapters.redis.ttl', $config['adapters']['redis']['ttl']);
25+
$container->setParameter('klp_mcp_server.adapters.redis.prefix', $config['adapters']['redis']['prefix'] ?? 'mcp_sse');
26+
$container->setParameter('klp_mcp_server.adapters.redis.connection', $config['adapters']['redis']['connection'] ?? 'default');
27+
$container->setParameter('klp_mcp_server.adapters.redis.ttl', $config['adapters']['redis']['ttl'] ?? 100);
2828
$container->setParameter('klp_mcp_server.tools', $config['tools']);
2929
$container->setParameter('klp_mcp_server.prompts', $config['prompts']);
3030
$container->setParameter('klp_mcp_server.resources', $config['resources']);

tests/DependencyInjection/KlpMcpServerExtensionTest.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,13 @@ public function test_load_sets_all_parameters_correctly(): void
3333
],
3434
'default_path' => '/default/path',
3535
'middlewares' => ['middleware1', 'middleware2'],
36+
'adapters' => [
37+
'redis' => [
38+
'prefix' => 'prefix',
39+
'connection' => 'localhost',
40+
'ttl' => 100,
41+
]
42+
],
3643
'server_provider' => 'sse',
3744
'sse_adapter' => 'custom_adapter',
3845
'tools' => [HelloWorldTool::class, VersionCheckTool::class],

0 commit comments

Comments
 (0)