Skip to content

Comments

Set idle timeout when streaming responses#52

Merged
mworrell merged 5 commits intomasterfrom
idle-timeout-resp-stream
Oct 2, 2025
Merged

Set idle timeout when streaming responses#52
mworrell merged 5 commits intomasterfrom
idle-timeout-resp-stream

Conversation

@mworrell
Copy link
Member

@mworrell mworrell commented Oct 2, 2025

Also upgrade cowboy, cowlib and zotonic_stdlib.
Add the lock file.

@mworrell mworrell requested review from Copilot and mmzeeman October 2, 2025 08:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds idle timeout management for streaming responses in cowmachine. The purpose is to prevent timeouts during streaming operations while maintaining appropriate timeout settings when not streaming.

  • Adds a configurable idle timeout mechanism for HTTP streaming responses
  • Sets idle timeout to infinity during streaming operations and restores it to 60 seconds afterward
  • Removes some commented-out code and upgrades dependencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mworrell mworrell requested a review from Copilot October 2, 2025 08:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 4 out of 7 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mworrell mworrell merged commit 55aa3e9 into master Oct 2, 2025
3 checks passed
@mworrell mworrell deleted the idle-timeout-resp-stream branch October 2, 2025 09:15
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