fix: update status code check for resource variable synchronization

jsbroks · jsbroks · commit 84fff2b6a337 · 2026-02-25T23:04:05.000-05:00
diff --git a/cmd/ctrlc/root/sync/pipe/pipe.go b/cmd/ctrlc/root/sync/pipe/pipe.go
@@ -277,7 +277,7 @@ func syncResourceVariables(
 				if varsResp.StatusCode() == 404 {
 					return fmt.Errorf("resource '%s' not found yet, retrying", resource.Identifier)
 				}
-				if varsResp.StatusCode() != 204 {
+				if varsResp.StatusCode() >= 400 {
 					return retry.Unrecoverable(
 						fmt.Errorf("failed to update resource variables for '%s': %s", resource.Identifier, string(varsResp.Body)),
 					)

Original file line number	Diff line number	Diff line change
`@@ -277,7 +277,7 @@ func syncResourceVariables(`
`277`	`277`	`if varsResp.StatusCode() == 404 {`
`278`	`278`	`return fmt.Errorf("resource '%s' not found yet, retrying", resource.Identifier)`
`279`	`279`	`}`
`280`		`- if varsResp.StatusCode() != 204 {`
	`280`	`+ if varsResp.StatusCode() >= 400 {`
`281`	`281`	`return retry.Unrecoverable(`
`282`	`282`	`fmt.Errorf("failed to update resource variables for '%s': %s", resource.Identifier, string(varsResp.Body)),`
`283`	`283`	`)`