fix: AWS status check within KES

the returned response body is not closed casued the file descriptor exceed the system limit
minio · Feb 10, 2024 · 0ed2a14 · 0ed2a14
1 parent 16a6929
commit 0ed2a14
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/internal/keystore/aws/secrets-manager.go b/internal/keystore/aws/secrets-manager.go
@@ -112,9 +112,12 @@ func (s *Store) Status(ctx context.Context) (kes.KeyStoreState, error) {
 	}
 
 	start := time.Now()
-	if _, err = http.DefaultClient.Do(req); err != nil {
+	res, err := http.DefaultClient.Do(req)
+	if err != nil {
 		return kes.KeyStoreState{}, &keystore.ErrUnreachable{Err: err}
 	}
+	defer res.Body.Close()
+
 	return kes.KeyStoreState{
 		Latency: time.Since(start),
 	}, nil