handle response nil, close response.Body

bluesky-social · Nov 22, 2024 · 6a25c33 · 6a25c33
1 parent b6f34c7
commit 6a25c33
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/bgs/handlers.go b/bgs/handlers.go
@@ -197,7 +197,10 @@ func (s *BGS) handleComAtprotoSyncRequestCrawl(ctx context.Context, body *comatp
 				for _, rpu := range s.nextCrawlers {
 					pu := rpu.JoinPath("/xrpc/com.atproto.sync.requestCrawl")
 					response, err := s.httpClient.Post(pu.String(), "application/json", bytes.NewReader(bodyBlob))
-					if err != nil {
+					if response != nil && response.Body != nil {
+						response.Body.Close()
+					}
+					if err != nil || response == nil {
 						log.Warnw("requestCrawl forward failed", "host", rpu, "err", err)
 					} else if response.StatusCode != http.StatusOK {
 						log.Warnw("requestCrawl forward failed", "host", rpu, "status", response.Status)