chore: delete user codersdk to support status code regression

Emyrk · Emyrk · commit 592d44463039 · 2024-08-05T13:34:17.000-05:00
diff --git a/codersdk/users.go b/codersdk/users.go
@@ -309,7 +309,11 @@ func (c *Client) DeleteUser(ctx context.Context, id uuid.UUID) error {
 		return err
 	}
 	defer res.Body.Close()
-	if res.StatusCode != http.StatusOK {
+	// Check for both status codes, there was a release that changed this response
+	// to StatusNoContent. To be compatible with that, the second condition is
+	// included.
+	// The 'http.StatusNoContent' check can be removed in 2025
+	if res.StatusCode != http.StatusOK && res.StatusCode != http.StatusNoContent {
 		return ReadBodyAsError(res)
 	}
 	return nil

Original file line number	Diff line number	Diff line change
`@@ -309,7 +309,11 @@ func (c *Client) DeleteUser(ctx context.Context, id uuid.UUID) error {`
`309`	`309`	`return err`
`310`	`310`	`}`
`311`	`311`	`defer res.Body.Close()`
`312`		`- if res.StatusCode != http.StatusOK {`
	`312`	`+ // Check for both status codes, there was a release that changed this response`
	`313`	`+ // to StatusNoContent. To be compatible with that, the second condition is`
	`314`	`+ // included.`
	`315`	`+ // The 'http.StatusNoContent' check can be removed in 2025`
	`316`	`+ if res.StatusCode != http.StatusOK && res.StatusCode != http.StatusNoContent {`
`313`	`317`	`return ReadBodyAsError(res)`
`314`	`318`	`}`
`315`	`319`	`return nil`