Fix logic to create/delete/update sensitive variables (#37)

* Fix logic to create/delete/update sensitive variables

Author: alfespa17

internal/provider/organization_variable_resource.go

@@ -172,16 +172,23 @@ func (r *OrganizationVariableResource) Create(ctx context.Context, req resource.
 	organizationVariable := &client.OrganizationVariableEntity{}
 
 	err = jsonapi.UnmarshalPayload(strings.NewReader(string(bodyResponse)), organizationVariable)
-
+	tflog.Info(ctx, string(bodyResponse))
 	if err != nil {
 		resp.Diagnostics.AddError("Error unmarshal payload response", fmt.Sprintf("Error unmarshal payload response: %s", err))
 		return
 	}
 
 	tflog.Info(ctx, "Body Response", map[string]any{"bodyResponse": string(bodyResponse)})
 
+	if *organizationVariable.Sensitive {
+		tflog.Info(ctx, "Variable value is not included in response, setting values the same as the plan for sensitive=true...")
+		plan.Value = types.StringValue(plan.Value.ValueString())
+	} else {
+		tflog.Info(ctx, "Variable value is included in response...")
+		plan.Value = types.StringValue(organizationVariable.Value)
+	}
+
 	plan.Key = types.StringValue(organizationVariable.Key)
-	plan.Value = types.StringValue(organizationVariable.Value)
 	plan.Description = types.StringValue(organizationVariable.Description)
 	plan.Category = types.StringValue(organizationVariable.Category)
 	plan.Sensitive = types.BoolValue(*organizationVariable.Sensitive)
@@ -231,8 +238,15 @@ func (r *OrganizationVariableResource) Read(ctx context.Context, req resource.Re
 
 	tflog.Info(ctx, "Body Response", map[string]any{"bodyResponse": string(bodyResponse)})
 
+	if *organizationVariable.Sensitive {
+		tflog.Info(ctx, "Variable value is not included in response, setting values the same as the current state value")
+		state.Value = types.StringValue(state.Value.ValueString())
+	} else {
+		tflog.Info(ctx, "Variable value is included in response...")
+		state.Value = types.StringValue(organizationVariable.Value)
+	}
+
 	state.Key = types.StringValue(organizationVariable.Key)
-	state.Value = types.StringValue(organizationVariable.Value)
 	state.Description = types.StringValue(organizationVariable.Description)
 	state.Category = types.StringValue(organizationVariable.Category)
 	state.Sensitive = types.BoolValue(*organizationVariable.Sensitive)
@@ -330,7 +344,15 @@ func (r *OrganizationVariableResource) Update(ctx context.Context, req resource.
 
 	plan.ID = types.StringValue(state.ID.ValueString())
 	plan.Key = types.StringValue(organizationVariable.Key)
-	plan.Value = types.StringValue(organizationVariable.Value)
+
+	if *organizationVariable.Sensitive {
+		tflog.Info(ctx, "Variable value is not included in response, setting values the same as the plan for sensitive=true...")
+		plan.Value = types.StringValue(plan.Value.ValueString())
+	} else {
+		tflog.Info(ctx, "Variable value is included in response...")
+		plan.Value = types.StringValue(organizationVariable.Value)
+	}
+
 	plan.Description = types.StringValue(organizationVariable.Description)
 	plan.Category = types.StringValue(organizationVariable.Category)
 	plan.Sensitive = types.BoolValue(*organizationVariable.Sensitive)

internal/provider/workspace_variable_resource.go

@@ -184,8 +184,15 @@ func (r *WorkspaceVariableResource) Create(ctx context.Context, req resource.Cre
 
 	tflog.Info(ctx, "Body Response", map[string]any{"bodyResponse": string(bodyResponse)})
 
+	if workspaceVariable.Sensitive {
+		tflog.Info(ctx, "Variable value is not included in response, setting values the same as the plan for sensitive=true...")
+		plan.Value = types.StringValue(plan.Value.ValueString())
+	} else {
+		tflog.Info(ctx, "Variable value is included in response...")
+		plan.Value = types.StringValue(workspaceVariable.Value)
+	}
+
 	plan.Key = types.StringValue(workspaceVariable.Key)
-	plan.Value = types.StringValue(workspaceVariable.Value)
 	plan.Description = types.StringValue(workspaceVariable.Description)
 	plan.Category = types.StringValue(workspaceVariable.Category)
 	plan.Sensitive = types.BoolValue(workspaceVariable.Sensitive)
@@ -235,8 +242,15 @@ func (r *WorkspaceVariableResource) Read(ctx context.Context, req resource.ReadR
 
 	tflog.Info(ctx, "Body Response", map[string]any{"bodyResponse": string(bodyResponse)})
 
+	if workspaceVariable.Sensitive {
+		tflog.Info(ctx, "Variable value is not included in response, setting values the same as the current state value")
+		state.Value = types.StringValue(state.Value.ValueString())
+	} else {
+		tflog.Info(ctx, "Variable value is included in response...")
+		state.Value = types.StringValue(workspaceVariable.Value)
+	}
+
 	state.Key = types.StringValue(workspaceVariable.Key)
-	state.Value = types.StringValue(workspaceVariable.Value)
 	state.Description = types.StringValue(workspaceVariable.Description)
 	state.Category = types.StringValue(workspaceVariable.Category)
 	state.Sensitive = types.BoolValue(workspaceVariable.Sensitive)
@@ -331,9 +345,16 @@ func (r *WorkspaceVariableResource) Update(ctx context.Context, req resource.Upd
 		return
 	}
 
+	if workspaceVariable.Sensitive {
+		tflog.Info(ctx, "Variable value is not included in response, setting values the same as the plan for sensitive=true...")
+		plan.Value = types.StringValue(plan.Value.ValueString())
+	} else {
+		tflog.Info(ctx, "Variable value is included in response...")
+		plan.Value = types.StringValue(workspaceVariable.Value)
+	}
+
 	plan.ID = types.StringValue(state.ID.ValueString())
 	plan.Key = types.StringValue(workspaceVariable.Key)
-	plan.Value = types.StringValue(workspaceVariable.Value)
 	plan.Description = types.StringValue(workspaceVariable.Description)
 	plan.Category = types.StringValue(workspaceVariable.Category)
 	plan.Sensitive = types.BoolValue(workspaceVariable.Sensitive)