Add errors for missing values on reads

nick-benoit · nick-benoit · commit ec0c547b700a · 2025-11-26T17:53:54.000-07:00
diff --git a/internal/kibana/security_list/create.go b/internal/kibana/security_list/create.go
@@ -55,6 +55,7 @@ func (r *securityListResource) Create(ctx context.Context, req resource.CreateRe
 
 	if readResp == nil || readResp.JSON200 == nil {
 		resp.State.RemoveResource(ctx)
+		resp.Diagnostics.AddError("Failed to fetch security list", "API returned empty response")
 		return
 	}
 
diff --git a/internal/kibana/security_list/update.go b/internal/kibana/security_list/update.go
@@ -61,6 +61,7 @@ func (r *securityListResource) Update(ctx context.Context, req resource.UpdateRe
 
 	if readResp == nil || readResp.JSON200 == nil {
 		resp.State.RemoveResource(ctx)
+		resp.Diagnostics.AddError("Failed to fetch security list", "API returned empty response")
 		return
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,7 @@ func (r *securityListResource) Create(ctx context.Context, req resource.CreateRe`
`55`	`55`
`56`	`56`	`if readResp == nil \|\| readResp.JSON200 == nil {`
`57`	`57`	`resp.State.RemoveResource(ctx)`
	`58`	`+ resp.Diagnostics.AddError("Failed to fetch security list", "API returned empty response")`
`58`	`59`	`return`
`59`	`60`	`}`
`60`	`61`
Original file line number	Diff line number	Diff line change
`@@ -61,6 +61,7 @@ func (r *securityListResource) Update(ctx context.Context, req resource.UpdateRe`
`61`	`61`
`62`	`62`	`if readResp == nil \|\| readResp.JSON200 == nil {`
`63`	`63`	`resp.State.RemoveResource(ctx)`
	`64`	`+ resp.Diagnostics.AddError("Failed to fetch security list", "API returned empty response")`
`64`	`65`	`return`
`65`	`66`	`}`
`66`	`67`