fix(read): implement read_response_codes and avoid post-creationg drift

JackSlateur · JackSlateur · commit a4b2a2cbc694 · 2026-07-17T22:40:09.000+02:00
ReadResponseCodes was never actually used
Also, improve the situation when oldSanitized is null (aka: we just
created the object)

Signed-off-by: Alexandre Bruyelles &lt;git@jack.fr.eu.org&gt;
diff --git a/internal/provider/curl_resource.go b/internal/provider/curl_resource.go
@@ -721,7 +721,7 @@ func (r *CurlResource) Read(ctx context.Context, req resource.ReadRequest, resp
 	}
 
 	// Drift detection
-	if oldSanitized != sanitizedResponse {
+	if !responseCodeChecker(data.ReadResponseCodes, httpResp.StatusCode) || (oldSanitized != "null" && oldSanitized != sanitizedResponse) {
 		tflog.Warn(ctx, "Drift detected: Response has changed, marking for recreation.")
 		data.DriftMarker = types.StringValue(time.Now().Format(time.RFC3339Nano))
 	} else {

Original file line number	Diff line number	Diff line change
`@@ -721,7 +721,7 @@ func (r *CurlResource) Read(ctx context.Context, req resource.ReadRequest, resp`
`721`	`721`	`}`
`722`	`722`
`723`	`723`	`// Drift detection`
`724`		`- if oldSanitized != sanitizedResponse {`
	`724`	`+ if !responseCodeChecker(data.ReadResponseCodes, httpResp.StatusCode) \|\| (oldSanitized != "null" && oldSanitized != sanitizedResponse) {`
`725`	`725`	`tflog.Warn(ctx, "Drift detected: Response has changed, marking for recreation.")`
`726`	`726`	`data.DriftMarker = types.StringValue(time.Now().Format(time.RFC3339Nano))`
`727`	`727`	`} else {`