Skip to content

Commit 82f999a

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 e7eabcf commit 82f999a

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
@@ -1022,6 +1022,9 @@ export function generateDockerCompose(
10221022
HTTPS_PROXY: `http://${networkConfig.squidIp}:${SQUID_PORT}`,
10231023
http_proxy: `http://${networkConfig.squidIp}:${SQUID_PORT}`,
10241024
https_proxy: `http://${networkConfig.squidIp}:${SQUID_PORT}`,
1025+
// Prevent curl health check from routing localhost through Squid
1026+
NO_PROXY: `localhost,127.0.0.1,::1`,
1027+
no_proxy: `localhost,127.0.0.1,::1`,
10251028
// Rate limiting configuration
10261029
...(config.rateLimitConfig && {
10271030
AWF_RATE_LIMIT_ENABLED: String(config.rateLimitConfig.enabled),

0 commit comments

Comments
 (0)