Skip to content

Commit 26ecee7

Browse files
authored
Merge pull request #3 from HJianBo/pop-client-session
fix: Pop client session if it failed to initialized
2 parents 577664d + 2138d87 commit 26ecee7

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/mcp/client/mqtt.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -186,11 +186,13 @@ async def after_initialize():
186186
if self.on_mcp_connect:
187187
self._task_group.start_soon(self.on_mcp_connect, self, server_name, ("ok", init_result))
188188
except Exception as e:
189+
self.client_sessions.pop(server_name)
189190
logging.error(f"Failed to initialize server {server_name}: {e}")
190191
await exit_stack.aclose()
191192
self._task_group.start_soon(after_initialize)
192193
logger.debug(f"after initialize: {server_name}")
193194
except McpError as exc:
195+
self.client_sessions.pop(server_name)
194196
logger.error(f"Failed to connect to MCP server: {exc}")
195197
if self.on_mcp_connect:
196198
self._task_group.start_soon(self.on_mcp_connect, self, server_name, ("error", McpError))

0 commit comments

Comments
 (0)