Skip to content

Fix Server.Stop double close bug#7

Merged
fabricekabongo merged 1 commit into
mainfrom
codex/trouver-et-corriger-un-bug
Jun 27, 2025
Merged

Fix Server.Stop double close bug#7
fabricekabongo merged 1 commit into
mainfrom
codex/trouver-et-corriger-un-bug

Conversation

@fabricekabongo
Copy link
Copy Markdown
Owner

Summary

  • prevent panic on repeated Server.Stop calls by using sync.Once
  • add regression test for Server.Stop idempotency

Testing

  • go test ./... (fails: fetching modules requires network access)

https://chatgpt.com/codex/tasks/task_e_685ed807775c8333a738590b04040379

@deepsource-io
Copy link
Copy Markdown
Contributor

deepsource-io Bot commented Jun 27, 2025

Here's the code health analysis summary for commits 2a49b6c..1827a57. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Go LogoGo❌ Failure
❗ 1 occurence introduced
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@fabricekabongo fabricekabongo merged commit 8fd5a46 into main Jun 27, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant