release runtime provider fix for paths
authorAlexander Strizhakov <alex.strizhakov@gmail.com>
Thu, 8 Apr 2021 12:45:31 +0000 (15:45 +0300)
committerAlexander Strizhakov <alex.strizhakov@gmail.com>
Thu, 8 Apr 2021 12:45:31 +0000 (15:45 +0300)
lib/pleroma/config/release_runtime_provider.ex
mix.exs
test/pleroma/config/release_runtime_provider_test.exs

index 46fa355591c77ad0b76d516d6dc4bafe2e6658e9..e5e9d3dcd2c2a1019d990e76bfdaa9abb154e589 100644 (file)
@@ -11,10 +11,11 @@ defmodule Pleroma.Config.ReleaseRuntimeProvider do
   def load(config, opts) do
     with_defaults = Config.Reader.merge(config, Pleroma.Config.Holder.release_defaults())
 
-    config_path = opts[:config_path]
+    config_path =
+      opts[:config_path] || System.get_env("PLEROMA_CONFIG_PATH") || "/etc/pleroma/config.exs"
 
     with_runtime_config =
-      if config_path && File.exists?(config_path) do
+      if File.exists?(config_path) do
         runtime_config = Config.Reader.read!(config_path)
 
         with_defaults
@@ -32,10 +33,14 @@ defmodule Pleroma.Config.ReleaseRuntimeProvider do
         with_defaults
       end
 
-    exported_config_path = opts[:exported_config_path]
+    exported_config_path =
+      opts[:exported_config_path] ||
+        config_path
+        |> Path.dirname()
+        |> Path.join("#{Pleroma.Config.get(:env)}.exported_from_db.secret.exs")
 
     with_exported =
-      if exported_config_path && File.exists?(exported_config_path) do
+      if File.exists?(exported_config_path) do
         exported_config = Config.Reader.read!(exported_config_path)
         Config.Reader.merge(with_runtime_config, exported_config)
       else
diff --git a/mix.exs b/mix.exs
index 7328b533b497739c4efc52277f2f26cbe2163e64..fe5d9d9637b5990945c62886809b91368612f0af 100644 (file)
--- a/mix.exs
+++ b/mix.exs
@@ -38,7 +38,7 @@ defmodule Pleroma.Mixfile do
           include_executables_for: [:unix],
           applications: [ex_syslogger: :load, syslog: :load, eldap: :transient],
           steps: [:assemble, &put_otp_version/1, &copy_files/1, &copy_nginx_config/1],
-          config_providers: [{Pleroma.Config.ReleaseRuntimeProvider, release_config_paths()}]
+          config_providers: [{Pleroma.Config.ReleaseRuntimeProvider, []}]
         ]
       ]
     ]
@@ -67,17 +67,6 @@ defmodule Pleroma.Mixfile do
     release
   end
 
-  defp release_config_paths do
-    config_path = System.get_env("PLEROMA_CONFIG_PATH") || "/etc/pleroma/config.exs"
-
-    exported_config_path =
-      config_path
-      |> Path.dirname()
-      |> Path.join("#{Mix.env()}.exported_from_db.secret.exs")
-
-    [config_path: config_path, exported_config_path: exported_config_path]
-  end
-
   # Configuration for the OTP application.
   #
   # Type `mix help compile.app` for more information.
index 1921698c5c40a4f295afa669665d0098fb2b3e4b..6578d32686fc9336c33e3edfb8b836c06319b171 100644 (file)
@@ -8,7 +8,6 @@ defmodule Pleroma.Config.ReleaseRuntimeProviderTest do
       ExUnit.CaptureIO.capture_io(fn ->
         merged = ReleaseRuntimeProvider.load([], [])
         assert merged == Pleroma.Config.Holder.release_defaults()
-        IO.inspect(merged)
       end) =~
         "!!! Config path is not declared! Please ensure it exists and that PLEROMA_CONFIG_PATH is unset or points to an existing file"
     end