Merge pull request #2890 from jmprusi/jmprusi/tunnel_retry

🐛 tunneler: return retry-after when dialer not ready
kcp-dev · Mar 14, 2023 · 46aed50 · 46aed50
2 parents bf1d290 + 46f7245
commit 46aed50
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/tunneler/podsubresourceproxy_handler.go b/pkg/tunneler/podsubresourceproxy_handler.go
@@ -252,7 +252,8 @@ func podSubresourceURL(downstreamNamespaceName, podName, subresource string) (*u
 func (tn *tunneler) Proxy(clusterName logicalcluster.Name, syncerName string, rw http.ResponseWriter, req *http.Request) {
 	d := tn.getDialer(clusterName, syncerName)
 	if d == nil || isClosedChan(d.Done()) {
-		http.Error(rw, "syncer tunnels: tunnel closed", http.StatusInternalServerError)
+		rw.Header().Set("Retry-After", "1")
+		http.Error(rw, "syncer tunnels: tunnel closed", http.StatusServiceUnavailable)
 		return
 	}