Skip to content

fix(gateway): resolve review issues for server concurrency and ipc hardening#2

Merged
fennoai[bot] merged 1 commit into
feat/gateway-step1-ipc-skeletonfrom
fork-pr-256-1776126585
Apr 14, 2026
Merged

fix(gateway): resolve review issues for server concurrency and ipc hardening#2
fennoai[bot] merged 1 commit into
feat/gateway-step1-ipc-skeletonfrom
fork-pr-256-1776126585

Conversation

@fennoai
Copy link
Copy Markdown

@fennoai fennoai Bot commented Apr 14, 2026

Requested by @pionxe

Summary

  • Fix WaitGroup Add/Wait race in gateway server connection lifecycle.
  • Add configurable connection cap with default limit and fast reject on overflow.
  • Add read/write deadlines to prevent slow-connection goroutine starvation.
  • Remove parallel execution in Windows ACL tests that override package-level hooks.
  • Add/extend tests for new defaults, timeout behavior, and connection-limit branches.

…rdening

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: pionxe <148670367+pionxe@users.noreply.github.com>
@fennoai
Copy link
Copy Markdown
Author

fennoai Bot commented Apr 14, 2026

Important

Configuration Required

No API key has been configured for this organization. XGopilot requires an API key to process requests.

Please visit the Settings page to configure your API key.

@fennoai fennoai Bot merged commit 1bd05cf into feat/gateway-step1-ipc-skeleton Apr 14, 2026
1 check passed
@fennoai fennoai Bot deleted the fork-pr-256-1776126585 branch April 14, 2026 00:35
@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

pionxe pushed a commit that referenced this pull request Apr 24, 2026
refactor(tui): 命令菜单/Activity/filepicker/progress 组件化并合并重复入口
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants