Skip to content

Commit 07dad56

Browse files
Mossakaclaude
andcommitted
fix(proxy): add NO_PROXY to api-proxy container for health check
The api-proxy container's curl health check was routing through Squid because of the newly-added lowercase http_proxy/https_proxy env vars. curl respects lowercase proxy vars, causing localhost health checks to fail. Add NO_PROXY/no_proxy with localhost entries to prevent this. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 35df06c commit 07dad56

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/docker-manager.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1024,6 +1024,9 @@ export function generateDockerCompose(
10241024
HTTPS_PROXY: `http://${networkConfig.squidIp}:${SQUID_PORT}`,
10251025
http_proxy: `http://${networkConfig.squidIp}:${SQUID_PORT}`,
10261026
https_proxy: `http://${networkConfig.squidIp}:${SQUID_PORT}`,
1027+
// Prevent curl health check from routing localhost through Squid
1028+
NO_PROXY: `localhost,127.0.0.1,::1`,
1029+
no_proxy: `localhost,127.0.0.1,::1`,
10271030
// Rate limiting configuration
10281031
...(config.rateLimitConfig && {
10291032
AWF_RATE_LIMIT_ENABLED: String(config.rateLimitConfig.enabled),

0 commit comments

Comments
 (0)