Skip to content

Conversation

@smatsuodev
Copy link
Owner

No description provided.

@smatsuodev smatsuodev self-assigned this Aug 20, 2025
Copy link
Collaborator

@harsssh harsssh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作確認してないけど良さそうに見える
ループしてる分遅そうだけど耐えて欲しい

@harsssh
Copy link
Collaborator

harsssh commented Aug 24, 2025

動作確認とコンフリクト解消します

@harsssh
Copy link
Collaborator

harsssh commented Aug 24, 2025

タイムアウト前にちゃんと終了したプロセスを kill() しようとしてる

@harsssh
Copy link
Collaborator

harsssh commented Aug 24, 2025

タイムアウト自体はできてそうです。
リクエストが来ない場合もOKそう。

$ nc localhost 8080
HTTP/1.1 408 Request Timeout
Content-Length: 0

@harsssh
Copy link
Collaborator

harsssh commented Aug 24, 2025

タイムアウト前にちゃんと終了したプロセスを kill() しようとしてる

ぼくが CgiProcessRepository から消してないのが悪い気がしてきた

@harsssh
Copy link
Collaborator

harsssh commented Aug 24, 2025

治しました

@harsssh harsssh merged commit 18d1f3b into main Aug 24, 2025
3 checks passed
@harsssh harsssh deleted the impl-timeout branch August 24, 2025 06:18
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.

3 participants