fix(mcp): McpServerController.start/stop race creates a second socket host and leaks the first
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
研究方向
Start with lib/core/mcp/mcp_server_controller.dart:102-124 and trace how start() and stop() handle overlapping calls and the host.changes subscription. Add tests for two concurrent start() calls creating one host and for stop() during startup ending in a stopped state; done when both behaviors pass and the subscription is cancelled on stop.
由索引模型根據 Issue 內容生成。
描述
Problem
McpServerController.start() (lib/core/mcp/mcp_server_controller.dart:102-124) checks _host != null first, then awaits _version() and host.start(), and assigns _host only after that. Two overlapping calls (startIfEnabled + setEnabled(true), repeated toggle, regenerateToken) create two McpSocketHosts: the first is leaked (never stopped), both write the same endpoint file and the token in the file may not belong to the host that is tracked.
stop() during an in-flight start() sees _host == null, does nothing, and the server comes up afterwards while the setting says disabled.
host.changes.listen is also never cancelled.
Expected
- Serialize start / stop (single in-flight future or a lock) and assign
_hostbefore the first await. - Cancel the
changessubscription on stop. - Test: concurrent
start()x2 yields one host;stop()duringstart()ends stopped.
- 主要語言
- Dart
- 星號
- 23
- 分支
- 1
- 平均合併
- 31 分鐘
- 30 天內合併 PR
- 570
環境準備
在瀏覽器裡用你自己的 GitHub 帳號啟動這個專案的開發容器。
- 沒有 Dockerfile 或 Docker Compose 檔案
- 有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
QueryaHub/Querya-Desktop 的其他 Issue
-
enhancement ui
難度 2/5 1-3 小時 新手友好度 78/100
QueryaHub/Querya-Desktop#1689 ·
維護者通常 1 天內回覆
-
ui(dialogs): make destructive query and DML preview dialogs adaptive and prevent table name overflow未關閉bug frontend
難度 2/5 1-3 小時 新手友好度 82/100
QueryaHub/Querya-Desktop#1642 ·
維護者通常 1 天內回覆
-
bug error-handling settings
難度 1/5 1-3 小時 新手友好度 78/100
QueryaHub/Querya-Desktop#1637 ·
維護者通常 1 天內回覆
-
enhancement
難度 1/5 1 小時以內 新手友好度 88/100
QueryaHub/Querya-Desktop#1636 ·
維護者通常 1 天內回覆
-
enhancement marketplace
難度 2/5 1-3 小時 新手友好度 78/100
QueryaHub/Querya-Desktop#1635 ·
維護者通常 1 天內回覆
查看 QueryaHub/Querya-Desktop 的全部 Issue
相似的 Issue
-
enhancement
難度 2/5 1-3 小時 新手友好度 77/100
atsign-foundation/at_server#2896 ·
維護者通常 1 天內回覆
-
bug
難度 2/5 1-3 小時 新手友好度 70/100
lugassawan/swe-workbench#823 ·
維護者通常 1 天內回覆
-
auto-implement
難度 2/5 1-3 小時 新手友好度 76/100
bannzai/kashakeibo#142 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 1/5 1-3 小時 新手友好度 82/100
meddouribadis/arclauncher#53 ·
維護者通常 1 天內回覆
-
p: material_ui package team-design
難度 2/5 1-3 小時 新手友好度 66/100
flutter/flutter#195096 · 1 則留言 ·
維護者通常 1 天內回覆