Skip to content

feat(config): Make Metrics Server timeouts configurable#114

Merged
allisson merged 14 commits intomainfrom
fix-metrics-server
Mar 7, 2026
Merged

feat(config): Make Metrics Server timeouts configurable#114
allisson merged 14 commits intomainfrom
fix-metrics-server

Conversation

@allisson
Copy link
Owner

@allisson allisson commented Mar 7, 2026

Previously, the Metrics Server used hardcoded timeout values (15s for Read/Write,
60s for Idle). This change introduces environment variables to allow these
timeouts to be configured, improving flexibility in different environments.

Changes:

  • Added METRICS_SERVER_READ_TIMEOUT_SECONDS (default: 15s)
  • Added METRICS_SERVER_WRITE_TIMEOUT_SECONDS (default: 15s)
  • Added METRICS_SERVER_IDLE_TIMEOUT_SECONDS (default: 60s)
  • Implemented validation for timeouts (1s to 300s range)
  • Updated NewDefaultMetricsServer to accept custom timeout values
  • Integrated configuration into the Dependency Injection container
  • Updated .env.example with new configuration options

allisson added 14 commits March 7, 2026 11:17
Previously, the Metrics Server used hardcoded timeout values (15s for Read/Write,
60s for Idle). This change introduces environment variables to allow these
timeouts to be configured, improving flexibility in different environments.

Changes:
- Added METRICS_SERVER_READ_TIMEOUT_SECONDS (default: 15s)
- Added METRICS_SERVER_WRITE_TIMEOUT_SECONDS (default: 15s)
- Added METRICS_SERVER_IDLE_TIMEOUT_SECONDS (default: 60s)
- Implemented validation for timeouts (1s to 300s range)
- Updated NewDefaultMetricsServer to accept custom timeout values
- Integrated configuration into the Dependency Injection container
- Updated .env.example with new configuration options
@allisson allisson merged commit 311de44 into main Mar 7, 2026
3 checks passed
@allisson allisson deleted the fix-metrics-server branch March 7, 2026 14:54
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.

1 participant