Skip to content

Conversation

@gammazero
Copy link
Contributor

Added a configurable fallback timeout for the gateway handler, defaulting to 1 hour. Configurable via MaxRequestDuration in the gateway config.

Replaces #1069

@gammazero gammazero requested review from a team and lidel as code owners December 13, 2025 03:56

// MaxRequestDuration per-request timeout for the gateway handler. It
// defaults to DefaultMaxRequestDuration.
MaxRequestDuration time.Duration
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Does this need to be a duration string that can be parsed (e.g. "1h")?

Added a configurable fallback timeout for the gateway handler, defaulting to 1 hour. Configurable via `MaxRequestDuration` in the gateway config.

Replaces #1069
@gammazero gammazero force-pushed the config-gw-request-timeout branch from 110dc2d to 8e2e90a Compare December 13, 2025 03:58
@codecov
Copy link

codecov bot commented Dec 13, 2025

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.09%. Comparing base (f76a2a7) to head (8e2e90a).

Files with missing lines Patch % Lines
gateway/gateway.go 50.00% 1 Missing and 1 partial ⚠️

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1079   +/-   ##
=======================================
  Coverage   61.09%   61.09%           
=======================================
  Files         264      264           
  Lines       26222    26226    +4     
=======================================
+ Hits        16020    16023    +3     
+ Misses       8524     8522    -2     
- Partials     1678     1681    +3     
Files with missing lines Coverage Δ
gateway/handler.go 77.91% <100.00%> (ø)
gateway/gateway.go 76.62% <50.00%> (-1.46%) ⬇️

... and 9 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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