disable keep alives for orchestrator proxy to match settings of current nginx based session-proxy

sitole · sitole · commit 709dd69c3006 · 2025-03-18T15:42:19.000+01:00
with new implementation there is error that envd returns "connection
reset by peer" when there was support for keep alive tcp connections for
re-use.
diff --git a/packages/orchestrator/internal/proxy/proxy.go b/packages/orchestrator/internal/proxy/proxy.go
@@ -69,7 +69,7 @@ func (p *SandboxProxy) Start() error {
 		IdleConnTimeout:       620 * time.Second, // Matches keepalive_timeout
 		TLSHandshakeTimeout:   10 * time.Second,  // Similar to client_header_timeout
 		ResponseHeaderTimeout: 24 * time.Hour,    // Matches proxy_read_timeout
-		DisableKeepAlives:     false,             // Allow keep-alive
+		DisableKeepAlives:     true,              // Disable keep-alives, envd doesn't support idle connections
 	}
 
 	p.server.Handler = http.HandlerFunc(p.proxyHandler(serverTransport))

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ func (p *SandboxProxy) Start() error {`
`69`	`69`	`IdleConnTimeout: 620 * time.Second, // Matches keepalive_timeout`
`70`	`70`	`TLSHandshakeTimeout: 10 * time.Second, // Similar to client_header_timeout`
`71`	`71`	`ResponseHeaderTimeout: 24 * time.Hour, // Matches proxy_read_timeout`
`72`		`- DisableKeepAlives: false, // Allow keep-alive`
	`72`	`+ DisableKeepAlives: true, // Disable keep-alives, envd doesn't support idle connections`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`p.server.Handler = http.HandlerFunc(p.proxyHandler(serverTransport))`