Fix linter issues in write-only implementation.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: devops-rob

internal/provider/curl_resource.go

@@ -591,7 +591,7 @@ func (r *CurlResource) Create(ctx context.Context, req resource.CreateRequest, r
 	}
 	data.RequestUrlString = types.StringValue(request.URL.String())
 
-	tflog.Debug(ctx, fmt.Sprintf("Resource create API Call: \nURL: %s\nHeaders: %s\nMethod: %s\nRequest Body: %s\n", request.URL.String(), request.Header, request.Method, requestBodyForLog(data.RequestBody, data.RequestBodyWo, usedWriteOnlyBody)))
+	tflog.Debug(ctx, fmt.Sprintf("Resource create API Call: \nURL: %s\nHeaders: %s\nMethod: %s\nRequest Body: %s\n", request.URL.String(), request.Header, request.Method, requestBodyForLog(data.RequestBody, usedWriteOnlyBody)))
 	timeout := 10 * time.Second
 	if !data.Timeout.IsNull() {
 		timeout = time.Duration(data.Timeout.ValueInt64()) * time.Second
@@ -726,7 +726,7 @@ func (r *CurlResource) executeReadRequest(ctx context.Context, data CurlResource
 		request.URL.RawQuery = params.Encode()
 	}
 
-	tflog.Debug(ctx, fmt.Sprintf("Resource read API Call: \nURL: %s\nHeaders: %s\nMethod: %s\nRequest Body: %s\n", request.URL.String(), request.Header, request.Method, requestBodyForLog(data.ReadRequestBody, data.ReadRequestBodyWo, usedWriteOnlyBody)))
+	tflog.Debug(ctx, fmt.Sprintf("Resource read API Call: \nURL: %s\nHeaders: %s\nMethod: %s\nRequest Body: %s\n", request.URL.String(), request.Header, request.Method, requestBodyForLog(data.ReadRequestBody, usedWriteOnlyBody)))
 
 	httpResp, err := client.Do(request)
 	if err != nil {
@@ -990,7 +990,7 @@ func (r *CurlResource) Delete(ctx context.Context, req resource.DeleteRequest, r
 	retryInterval := time.Duration(data.DestroyRetryInterval.ValueInt64()) * time.Second
 	maxRetry := int(data.DestroyMaxRetry.ValueInt64())
 
-	tflog.Debug(ctx, fmt.Sprintf("Resource destroy API Call: \nURL: %s\nHeaders: %s\nMethod: %s\nRequest Body: %s\n", request.URL.String(), request.Header, request.Method, requestBodyForLog(data.DestroyRequestBody, data.DestroyRequestBodyWo, usedWriteOnlyBody)))
+	tflog.Debug(ctx, fmt.Sprintf("Resource destroy API Call: \nURL: %s\nHeaders: %s\nMethod: %s\nRequest Body: %s\n", request.URL.String(), request.Header, request.Method, requestBodyForLog(data.DestroyRequestBody, usedWriteOnlyBody)))
 
 	var bodyBytes []byte
 	var statusCode int

internal/provider/write_only.go

@@ -85,7 +85,7 @@ func applyRequestHeadersWithWriteOnly(req *http.Request, headers, headersWo type
 	applyRequestHeaders(req, headersWo)
 }
 
-func requestBodyForLog(regular, writeOnly types.String, usedWriteOnly bool) string {
+func requestBodyForLog(regular types.String, usedWriteOnly bool) string {
 	if usedWriteOnly {
 		return "<redacted write-only request body>"
 	}
@@ -197,24 +197,6 @@ func writeOnlyVersionsChanged(state, config CurlResourceModel) bool {
 		state.DestroyRequestBodyWoVersion.ValueInt64() != config.DestroyRequestBodyWoVersion.ValueInt64()
 }
 
-func applyWriteOnlySnapshot(snapshot writeOnlyPrivateSnapshot, data *CurlResourceModel) {
-	if data == nil {
-		return
-	}
-	data.HeadersWo = stringMapToTypesMap(snapshot.HeadersCreate)
-	data.ReadHeadersWo = stringMapToTypesMap(snapshot.HeadersRead)
-	data.DestroyHeadersWo = stringMapToTypesMap(snapshot.HeadersDestroy)
-	if snapshot.BodyCreate != "" {
-		data.RequestBodyWo = types.StringValue(snapshot.BodyCreate)
-	}
-	if snapshot.BodyRead != "" {
-		data.ReadRequestBodyWo = types.StringValue(snapshot.BodyRead)
-	}
-	if snapshot.BodyDestroy != "" {
-		data.DestroyRequestBodyWo = types.StringValue(snapshot.BodyDestroy)
-	}
-}
-
 func nullWriteOnlyAttributes(data *CurlResourceModel) {
 	if data == nil {
 		return

internal/provider/write_only_test.go

@@ -108,7 +108,8 @@ func TestWriteOnlyPrivateSnapshotRoundtrip(t *testing.T) {
 		t.Fatalf("load failed: %v", diags)
 	}
 
-	if loaded.DestroyHeadersWo.Elements()["Authorization"].(types.String).ValueString() != "Bearer destroy" {
+	auth, ok := loaded.DestroyHeadersWo.Elements()["Authorization"].(types.String)
+	if !ok || auth.ValueString() != "Bearer destroy" {
 		t.Fatalf("unexpected destroy headers %#v", loaded.DestroyHeadersWo)
 	}
 	if loaded.ReadRequestBodyWo.ValueString() != `{"read":true}` {
@@ -149,7 +150,7 @@ func TestWriteOnlyVersionsChanged(t *testing.T) {
 }
 
 func TestRequestBodyForLogRedactsWriteOnly(t *testing.T) {
-	got := requestBodyForLog(types.StringNull(), types.StringValue("secret"), true)
+	got := requestBodyForLog(types.StringNull(), true)
 	if got != "<redacted write-only request body>" {
 		t.Fatalf("unexpected log value %q", got)
 	}