fix(all): remove potential nil pointer (#2862)

JoeWang1127 · web-flow · commit a8c6a7e8f14c · 2025-11-11T15:43:15.000-05:00
Remove the potential nil pointer call. Fixes #2863
diff --git a/internal/github/github.go b/internal/github/github.go
@@ -48,7 +48,12 @@ func (t *retryableTransport) RoundTrip(req *http.Request) (*http.Response, error
 		if err == nil && resp.StatusCode != http.StatusServiceUnavailable {
 			return resp, nil
 		}
-		slog.Warn("retrying due to error", "err", err, "status_code", resp.StatusCode)
+		if err != nil {
+			slog.Warn("retrying due to error", "err", err)
+		} else {
+			slog.Warn("retrying due to status code", "status_code", resp.StatusCode)
+		}
+
 		time.Sleep(retryDelay)
 	}
 	return resp, err

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,12 @@ func (t retryableTransport) RoundTrip(req http.Request) (*http.Response, error`
`48`	`48`	`if err == nil && resp.StatusCode != http.StatusServiceUnavailable {`
`49`	`49`	`return resp, nil`
`50`	`50`	`}`
`51`		`- slog.Warn("retrying due to error", "err", err, "status_code", resp.StatusCode)`
	`51`	`+ if err != nil {`
	`52`	`+ slog.Warn("retrying due to error", "err", err)`
	`53`	`+ } else {`
	`54`	`+ slog.Warn("retrying due to status code", "status_code", resp.StatusCode)`
	`55`	`+ }`
	`56`	`+`
`52`	`57`	`time.Sleep(retryDelay)`
`53`	`58`	`}`
`54`	`59`	`return resp, err`