Fix BitBucket status update panic

stefanprodan · stefanprodan · commit 32c59b2474c7 · 2024-01-30T15:23:38.000+02:00
Signed-off-by: Stefan Prodan &lt;stefan.prodan@gmail.com&gt;
diff --git a/internal/notifier/bitbucketserver.go b/internal/notifier/bitbucketserver.go
@@ -192,10 +192,10 @@ func (b BitbucketServer) duplicateBitbucketServerStatus(ctx context.Context, rev
 
 	// Make a GET call
 	d, err := b.Client.Do(req)
-	if err != nil && d.StatusCode != http.StatusNotFound {
+	if err != nil {
 		return false, fmt.Errorf("failed api call to check duplicate commit status: %w", err)
 	}
-	if isError(d) && d.StatusCode != http.StatusNotFound {
+	if d != nil && isError(d) && d.StatusCode != http.StatusNotFound {
 		defer d.Body.Close()
 		return false, fmt.Errorf("failed api call to check duplicate commit status: %d - %s", d.StatusCode, http.StatusText(d.StatusCode))
 	}

Original file line number	Diff line number	Diff line change
`@@ -192,10 +192,10 @@ func (b BitbucketServer) duplicateBitbucketServerStatus(ctx context.Context, rev`
`192`	`192`
`193`	`193`	`// Make a GET call`
`194`	`194`	`d, err := b.Client.Do(req)`
`195`		`- if err != nil && d.StatusCode != http.StatusNotFound {`
	`195`	`+ if err != nil {`
`196`	`196`	`return false, fmt.Errorf("failed api call to check duplicate commit status: %w", err)`
`197`	`197`	`}`
`198`		`- if isError(d) && d.StatusCode != http.StatusNotFound {`
	`198`	`+ if d != nil && isError(d) && d.StatusCode != http.StatusNotFound {`
`199`	`199`	`defer d.Body.Close()`
`200`	`200`	`return false, fmt.Errorf("failed api call to check duplicate commit status: %d - %s", d.StatusCode, http.StatusText(d.StatusCode))`
`201`	`201`	`}`