20200: Fix defer mistake
authorPeter Amstutz <peter.amstutz@curii.com>
Mon, 6 Mar 2023 17:47:59 +0000 (12:47 -0500)
committerPeter Amstutz <peter.amstutz@curii.com>
Mon, 6 Mar 2023 17:47:59 +0000 (12:47 -0500)
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>

lib/controller/handler.go

index 9830421eb24ac60edc45e2c0be29f16a78c3e08a..5d6bd9570c813e0f2eb7c721d0b5e02648fb452d 100644 (file)
@@ -194,7 +194,7 @@ func (h *Handler) limitLogCreateRequests(w http.ResponseWriter, req *http.Reques
        if cap(h.limitLogCreate) > 0 && req.Method == http.MethodPost && strings.HasPrefix(req.URL.Path, "/arvados/v1/logs") {
                select {
                case h.limitLogCreate <- struct{}{}:
-                       defer <-h.limitLogCreate
+                       defer func() { <-h.limitLogCreate }()
                        next.ServeHTTP(w, req)
                default:
                        http.Error(w, "Excess log messages", http.StatusServiceUnavailable)