astronomer / astro-cli / 801b5d85-8425-4cdc-86a7-604baa7d8910

36%

main: 36%

Build Type

Pull #2026

circleci

Committed by jlaneve

Commit Message

fix: proxy server hardening and review fixes

- Add HTTP server timeouts (ReadHeaderTimeout, WriteTimeout, IdleTimeout)
  to prevent slowloris attacks and goroutine leaks
- Add graceful shutdown via SIGTERM/SIGINT signal handler with 5s grace
- Cache reverse proxy instances per backend port with shared transport
  instead of allocating a new one per request
- Store CLI version in PID file and restart daemon on version mismatch
  to prevent incompatibilities after CLI upgrades
- Log warnings in removeProxyRoute instead of silently swallowing errors
- Inline trivial proxyEnabled() wrapper
- Fix readRoutes to handle whitespace-only routes files gracefully

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Pull Request Pull Request #2026: feat: built-in reverse proxy for multi-instance local development

Run Details

457 of 896 new or added lines in 10 files covered. (51.0%)

1 existing line in 1 file now uncovered.

24288 of 67629 relevant lines covered (35.91%)

8.57 hits per line