1 files changed, 71 insertions, 41 deletions
diff --git a/lib/mix/tasks/pleroma/config.ex b/lib/mix/tasks/pleroma/config.ex
index 590c7a914..b69c49b45 100644
--- a/lib/mix/tasks/pleroma/config.ex
+++ b/lib/mix/tasks/pleroma/config.ex
@@ -9,27 +9,28 @@ defmodule Mix.Tasks.Pleroma.Config do
   alias Pleroma.Web.AdminAPI.Config
   @shortdoc "Manages the location of the config"
   @moduledoc File.read!("docs/administration/CLI_tasks/config.md")
-  def run(["migrate_to_db"]) do
-    start_pleroma()
-
-    if Pleroma.Config.get([:instance, :dynamic_configuration]) do
-      Application.get_all_env(:pleroma)
-      |> Enum.reject(fn {k, _v} -> k in [Pleroma.Repo, :env] end)
-      |> Enum.each(fn {k, v} ->
-        key = to_string(k) |> String.replace("Elixir.", "")
 
-        key =
-          if String.starts_with?(key, "Pleroma.") do
-            key
-          else
-            ":" <> key
-          end
+  @groups [
+    :pleroma,
+    :logger,
+    :quack,
+    :mime,
+    :tesla,
+    :phoenix,
+    :cors_plug,
+    :auto_linker,
+    :esshd,
+    :ueberauth,
+    :http_signatures,
+    :web_push_encryption,
+    :joken
+  ]
 
-        {:ok, _} = Config.update_or_create(%{group: "pleroma", key: key, value: v})
-        Mix.shell().info("#{key} is migrated.")
-      end)
+  def run(["migrate_to_db"]) do
+    start_pleroma()
 
-      Mix.shell().info("Settings migrated.")
+    if Pleroma.Config.get([:configurable_from_database]) do
+      Enum.each(@groups, &load_and_create(&1))
     else
       Mix.shell().info(
         "Migration is not allowed by config. You can change this behavior in instance settings."
@@ -37,38 +38,67 @@ defmodule Mix.Tasks.Pleroma.Config do
     end
   end
 
-  def run(["migrate_from_db", env, delete?]) do
+  def run(["migrate_from_db" | options]) do
     start_pleroma()
 
-    delete? = if delete? == "true", do: true, else: false
-
-    if Pleroma.Config.get([:instance, :dynamic_configuration]) do
-      config_path = "config/#{env}.exported_from_db.secret.exs"
+    {opts, _} =
+      OptionParser.parse!(options,
+        strict: [env: :string, delete_from_db: :boolean],
+        aliases: [d: :delete_from_db]
+      )
 
-      {:ok, file} = File.open(config_path, [:write, :utf8])
+    with {:active?, true} <-
+           {:active?, Pleroma.Config.get([:configurable_from_database])},
+         env_path when is_binary(env_path) <- opts[:env],
+         config_path <- "config/#{env_path}.exported_from_db.secret.exs",
+         {:ok, file} <- File.open(config_path, [:write, :utf8]) do
       IO.write(file, "use Mix.Config\r\n")
 
-      Repo.all(Config)
-      |> Enum.each(fn config ->
-        IO.write(
-          file,
-          "config :#{config.group}, #{config.key}, #{
-            inspect(Config.from_binary(config.value), limit: :infinity)
-          }\r\n\r\n"
-        )
-
-        if delete? do
-          {:ok, _} = Repo.delete(config)
-          Mix.shell().info("#{config.key} deleted from DB.")
-        end
-      end)
+      Config
+      |> Repo.all()
+      |> Enum.each(&write_to_file_with_deletion(&1, file, opts[:delete_from_db]))
 
       File.close(file)
       System.cmd("mix", ["format", config_path])
     else
-      Mix.shell().info(
-        "Migration is not allowed by config. You can change this behavior in instance settings."
-      )
+      {:active?, false} ->
+        Mix.shell().info(
+          "migration is not allowed by config. You can change this behavior in instance settings."
+        )
+
+      error ->
+        Mix.shell().info("error occuried while opening file. #{inspect(error)}")
+    end
+  end
+
+  defp load_and_create(group) do
+    group
+    |> Application.get_all_env()
+    |> Enum.reject(fn {k, _v} ->
+      k in [Pleroma.Repo, :env, :configurable_from_database] or
+        (group == :phoenix and k == :serve_endpoints)
+    end)
+    |> Enum.each(fn {key, value} ->
+      key = inspect(key)
+      {:ok, _} = Config.update_or_create(%{group: inspect(group), key: key, value: value})
+
+      Mix.shell().info("settings for key #{key} migrated.")
+    end)
+
+    Mix.shell().info("settings for group :#{group} migrated.")
+  end
+
+  defp write_to_file_with_deletion(config, file, with_deletion) do
+    IO.write(
+      file,
+      "config #{config.group}, #{config.key}, #{
+        inspect(Config.from_binary(config.value), limit: :infinity)
+      }\r\n\r\n"
+    )
+
+    if with_deletion do
+      {:ok, _} = Repo.delete(config)
+      Mix.shell().info("#{config.key} deleted from DB.")
     end
   end
 end