remove anonymous function from plug
authorFloatingGhost <hannah@coffee-and-dreams.uk>
Thu, 14 Jul 2022 10:17:14 +0000 (11:17 +0100)
committerFloatingGhost <hannah@coffee-and-dreams.uk>
Thu, 14 Jul 2022 10:17:14 +0000 (11:17 +0100)
lib/pleroma/web/plugs/frontend_static.ex
lib/pleroma/web/swagger.ex

index 5d9ae2ff7db9d3adbac209900fb842c8c2502cbc..40f51e149a0da9d798ae81a1a84d977c47e5d4ab 100644 (file)
@@ -31,13 +31,13 @@ defmodule Pleroma.Web.Plugs.FrontendStatic do
     |> Keyword.put(:from, "__unconfigured_frontend_static_plug")
     |> Plug.Static.init()
     |> Map.put(:frontend_type, opts[:frontend_type])
-    |> Map.put(:if, Keyword.get(opts, :if, fn -> true end))
+    |> Map.put(:if, Keyword.get(opts, :if, true))
   end
 
   def call(conn, opts) do
     with false <- api_route?(conn.path_info),
          false <- invalid_path?(conn.path_info),
-         true <- opts[:if].(),
+         true <- enabled?(opts[:if]),
          frontend_type <- Map.get(opts, :frontend_type, :primary),
          path when not is_nil(path) <- file_path("", frontend_type) do
       call_static(conn, opts, path)
@@ -47,6 +47,10 @@ defmodule Pleroma.Web.Plugs.FrontendStatic do
     end
   end
 
+  defp enabled?(if_opt) when is_function(if_opt), do: if_opt.()
+  defp enabled?(true), do: true
+  defp enabled?(_), do: false
+
   defp invalid_path?(list) do
     invalid_path?(list, :binary.compile_pattern(["/", "\\", ":", "\0"]))
   end
index e75ce2180acfd6e78f8ada23c6339d94fb658e4a..0e32c468b226b4cf8fdf701ae87803a187196dfe 100644 (file)
@@ -2,6 +2,6 @@ defmodule Pleroma.Web.Swagger do
   alias Pleroma.Config
 
   def ui_enabled? do
-    Config.get([:frontends, :swagger, "enabled"])
+    Config.get([:frontends, :swagger, "enabled"], false)
   end
 end