[arvados] updated: 2.5.0-228-gfe738b30d

git repository hosting git at public.arvados.org
Mon Mar 6 17:48:13 UTC 2023


Summary of changes:
 lib/controller/handler.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  fe738b30d51dadb2b42ce7ae00b333d7ea157495 (commit)
      from  184d9fee6b72520be286b375ac569702246e224e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit fe738b30d51dadb2b42ce7ae00b333d7ea157495
Author: Peter Amstutz <peter.amstutz at curii.com>
Date:   Mon Mar 6 12:47:59 2023 -0500

    20200: Fix defer mistake
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>

diff --git a/lib/controller/handler.go b/lib/controller/handler.go
index 9830421eb..5d6bd9570 100644
--- a/lib/controller/handler.go
+++ b/lib/controller/handler.go
@@ -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)

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list