Sfoglia il codice sorgente

Merge pull request #19420 from clnperez/close-notify-fix

Move closeNotify to fix panic with newer golang
Brian Goff 9 anni fa
parent
commit
b6be645197
1 ha cambiato i file con 2 aggiunte e 1 eliminazioni
  1. 2 1
      api/server/router/build/build_routes.go

+ 2 - 1
api/server/router/build/build_routes.go

@@ -227,10 +227,11 @@ func (br *buildRouter) postBuild(ctx context.Context, w http.ResponseWriter, r *
 	if closeNotifier, ok := w.(http.CloseNotifier); ok {
 	if closeNotifier, ok := w.(http.CloseNotifier); ok {
 		finished := make(chan struct{})
 		finished := make(chan struct{})
 		defer close(finished)
 		defer close(finished)
+		clientGone := closeNotifier.CloseNotify()
 		go func() {
 		go func() {
 			select {
 			select {
 			case <-finished:
 			case <-finished:
-			case <-closeNotifier.CloseNotify():
+			case <-clientGone:
 				logrus.Infof("Client disconnected, cancelling job: build")
 				logrus.Infof("Client disconnected, cancelling job: build")
 				b.Cancel()
 				b.Cancel()
 			}
 			}