Merge pull request #63 from Infisical/daniel/fix-project-drift

fix: project role drift when no conditions are set
Infisical · Oct 9, 2024 · 2a1a088 · 2a1a088
2 parents 980fe51 + aca6fe9
commit 2a1a088
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/internal/provider/resource/project_role_resource.go b/internal/provider/resource/project_role_resource.go
@@ -301,13 +301,21 @@ func (r *projectRoleResource) Read(ctx context.Context, req resource.ReadRequest
 			}
 		}
 
-		permissionPlan = append(permissionPlan, projectRoleResourcePermissions{
-			Action:  types.StringValue(action),
-			Subject: types.StringValue(subject),
-			Conditions: &projectRoleResourcePermissionCondition{
+		var conditions *projectRoleResourcePermissionCondition
+
+		if el["conditions"] == nil {
+			conditions = nil
+		} else {
+			conditions = &projectRoleResourcePermissionCondition{
 				Environment: types.StringValue(environment),
 				SecretPath:  types.StringValue(secretPath),
-			},
+			}
+		}
+
+		permissionPlan = append(permissionPlan, projectRoleResourcePermissions{
+			Action:     types.StringValue(action),
+			Subject:    types.StringValue(subject),
+			Conditions: conditions,
 		})
 	}