Merge pull request #43 from mittwald/bugfix/app-userinputs-str

Fix double-quoting of app user inputs
mittwald · Apr 15, 2024 · 8a82e45 · 8a82e45
2 parents ed794b9 + ce80288
commit 8a82e45
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/internal/provider/resource/appresource/model_api.go b/internal/provider/resource/appresource/model_api.go
@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	"github.com/hashicorp/terraform-plugin-framework/diag"
+	"github.com/hashicorp/terraform-plugin-framework/path"
 	"github.com/hashicorp/terraform-plugin-framework/types"
 	"github.com/hashicorp/terraform-plugin-framework/types/basetypes"
 	"github.com/mittwald/terraform-provider-mittwald/api/mittwaldv2"
@@ -57,10 +58,14 @@ func (m *ResourceModel) ToCreateRequest(ctx context.Context, d diag.Diagnostics,
 	}
 
 	for key, value := range m.UserInputs.Elements() {
-		b.UserInputs = append(b.UserInputs, mittwaldv2.DeMittwaldV1AppSavedUserInput{
-			Name:  key,
-			Value: value.String(),
-		})
+		if s, ok := value.(types.String); ok {
+			b.UserInputs = append(b.UserInputs, mittwaldv2.DeMittwaldV1AppSavedUserInput{
+				Name:  key,
+				Value: s.ValueString(),
+			})
+		} else {
+			d.AddAttributeError(path.Root("user_inputs").AtMapKey(key), "invalid type", fmt.Sprintf("expected string, got %T", value))
+		}
 	}
 
 	return