Merge branch 'consent-ref-set-empty-fix' into 'master'

State inconsistency issue fix for empty consent_refs See merge request cidaas-management/terraform!127
Cidaas · Jan 3, 2025 · 4d67af0 · 4d67af0
2 parents c101ec0 + 35ce470
commit 4d67af0
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,11 @@
 ## Changelog
 
+### 3.3.8
+
+#### Bugfix
+
+- Fixed issue where empty `consent_refs` array was being omitted from API requests causing state inconsistency
+
 ### 3.3.7
 
 #### Enhancements

diff --git a/helpers/cidaas/app.go b/helpers/cidaas/app.go
@@ -79,7 +79,7 @@ type AppModel struct {
 	AllowedMfa                       []string                    `json:"allowed_mfa,omitempty"`
 	CaptchaRef                       string                      `json:"captcha_ref,omitempty"`
 	CaptchaRefs                      []string                    `json:"captcha_refs,omitempty"`
-	ConsentRefs                      []string                    `json:"consent_refs,omitempty"`
+	ConsentRefs                      []string                    `json:"consent_refs"`
 	CommunicationMediumVerification  string                      `json:"communication_medium_verification,omitempty"`
 	EmailVerificationRequired        *bool                       `json:"email_verification_required,omitempty"`
 	MobileNumberVerificationRequired *bool                       `json:"mobile_number_verification_required,omitempty"`

diff --git a/internal/resources/app_schema.go b/internal/resources/app_schema.go
@@ -566,6 +566,8 @@ var resourceAppSchema = schema.Schema{
 		"consent_refs": schema.SetAttribute{
 			ElementType: types.StringType,
 			Optional:    true,
+			Computed:    true,
+			Default:     setdefault.StaticValue(types.SetValueMust(types.StringType, []attr.Value{})),
 		},
 		"communication_medium_verification": schema.StringAttribute{
 			Optional: true,