From 6d14618c38be115f67056061b152d20b7c31fcdc Mon Sep 17 00:00:00 2001
From: Ramon Snir <r@mon.dev>
Date: Wed, 13 Nov 2024 09:30:53 -0500
Subject: [PATCH] no default function region: use site's current region

---
 internal/provider/site_build_settings_resource.go | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/internal/provider/site_build_settings_resource.go b/internal/provider/site_build_settings_resource.go
index e3b1145..9165b5b 100644
--- a/internal/provider/site_build_settings_resource.go
+++ b/internal/provider/site_build_settings_resource.go
@@ -167,7 +167,9 @@ func (r *siteBuildSettingsResource) Schema(_ context.Context, _ resource.SchemaR
 			"functions_region": schema.StringAttribute{
 				Optional: true,
 				Computed: true,
-				Default:  stringdefault.StaticString("us-east-2"),
+				PlanModifiers: []planmodifier.String{
+					stringplanmodifier.UseStateForUnknown(),
+				},
 			},
 			"pretty_urls": schema.BoolAttribute{
 				Optional: true,
@@ -303,7 +305,6 @@ func (r *siteBuildSettingsResource) write(ctx context.Context, plan *siteBuildSe
 	skipPrs := !plan.DeployPreviews.ValueBool()
 
 	site := netlifyapi.PartialSite{
-		FunctionsRegion: plan.FunctionsRegion.ValueStringPointer(),
 		BuildSettings: &netlifyapi.Repo{
 			Base:            plan.BaseDirectory.ValueStringPointer(),
 			PackagePath:     plan.PackageDirectory.ValueStringPointer(),
@@ -331,6 +332,15 @@ func (r *siteBuildSettingsResource) write(ctx context.Context, plan *siteBuildSe
 		site.BuildImage = plan.BuildImage.ValueStringPointer()
 	}
 
+	if plan.FunctionsRegion.IsUnknown() {
+		plan.FunctionsRegion = curState.FunctionsRegion
+	}
+	if plan.FunctionsRegion.IsNull() {
+		site.FunctionsRegion = curState.FunctionsRegion.ValueStringPointer()
+	} else {
+		site.FunctionsRegion = plan.FunctionsRegion.ValueStringPointer()
+	}
+
 	_, _, err := r.data.client.SitesAPI.
 		UpdateSite(ctx, plan.SiteID.ValueString()).
 		PartialSite(site).