Fix statuscode check for internal trigger error (#228)

gaborszakacs · web-flow · commit 12486fd5cd96 · 2025-08-22T09:42:04.000+02:00
diff --git a/bitriseapi/bitriseapi.go b/bitriseapi/bitriseapi.go
@@ -209,8 +209,8 @@ func TriggerBuild(url *url.URL, apiToken string, params TriggerAPIParamsModel, i
 	}
 	bodyString := string(body)
 
-	if resp.StatusCode >= 500 || resp.StatusCode < 600 {
-		return TriggerAPIResponseModel{}, false, errors.Wrapf(err, "TriggerBuild (url:%s): request sent, but received a server error response (http-code:%d, response body:%s)", url.String(), resp.StatusCode, bodyString)
+	if resp.StatusCode >= 500 && resp.StatusCode < 600 {
+		return TriggerAPIResponseModel{}, false, errors.New(fmt.Sprintf("TriggerBuild (url:%s): request sent, but received a server error response (http-code:%d, response body:%s)", url.String(), resp.StatusCode, bodyString))
 	}
 
 	var respModel TriggerAPIResponseModel

Original file line number	Diff line number	Diff line change
`@@ -209,8 +209,8 @@ func TriggerBuild(url *url.URL, apiToken string, params TriggerAPIParamsModel, i`
`209`	`209`	`}`
`210`	`210`	`bodyString := string(body)`
`211`	`211`
`212`		`- if resp.StatusCode >= 500 \|\| resp.StatusCode < 600 {`
`213`		`- return TriggerAPIResponseModel{}, false, errors.Wrapf(err, "TriggerBuild (url:%s): request sent, but received a server error response (http-code:%d, response body:%s)", url.String(), resp.StatusCode, bodyString)`
	`212`	`+ if resp.StatusCode >= 500 && resp.StatusCode < 600 {`
	`213`	`+ return TriggerAPIResponseModel{}, false, errors.New(fmt.Sprintf("TriggerBuild (url:%s): request sent, but received a server error response (http-code:%d, response body:%s)", url.String(), resp.StatusCode, bodyString))`
`214`	`214`	`}`
`215`	`215`
`216`	`216`	`var respModel TriggerAPIResponseModel`