Make oidc provider transfer more robust (#1237)

pluralsh · Sep 5, 2023 · 0676471 · 0676471
1 parent bcbb181
commit 0676471
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/apps/core/lib/core/services/clusters/transfer.ex b/apps/core/lib/core/services/clusters/transfer.ex
@@ -78,8 +78,12 @@ defmodule Core.Services.Clusters.Transfer do
 
   defp maybe_reparent_provider(xact, %OIDCProvider{installation_id: inst_id} = provider) do
     add_operation(xact, {:oidc, inst_id}, fn %{^inst_id => inst} ->
-      OIDCProvider.changeset(provider, %{installation_id: inst.id})
-      |> Core.Repo.update()
+      case Core.Repo.get_by(OIDCProvider, installation_id: inst.id) do
+        %OIDCProvider{} = prov -> {:ok, prov}
+        _ ->
+          OIDCProvider.changeset(provider, %{installation_id: inst.id})
+          |> Core.Repo.update()
+      end
     end)
   end
   defp maybe_reparent_provider(xact, _), do: xact

diff --git a/docker-compose.yml b/docker-compose.yml
@@ -1,7 +1,7 @@
 version: '3'
 services:
   db:
-    image: postgres:latest
+    image: postgres:14
     restart: always
     ports:
     - 5432:5432