Adds "invite_only" disable_registration config option

config/runtime.exs

@@ -134,7 +134,6 @@ enable_email_verification =
 disable_registration =
   config_dir
   |> get_var_from_path_or_env("DISABLE_REGISTRATION", "false")
-  |> String.to_existing_atom()
 
 hcaptcha_sitekey = get_var_from_path_or_env(config_dir, "HCAPTCHA_SITEKEY")
 hcaptcha_secret = get_var_from_path_or_env(config_dir, "HCAPTCHA_SECRET")
@@ -200,7 +199,7 @@ config :plausible,
 config :plausible, :selfhost,
   disable_authentication: disable_auth,
   enable_email_verification: enable_email_verification,
-  disable_registration: if(!disable_auth, do: disable_registration, else: false)
+  disable_registration: if(!disable_auth, do: disable_registration, else: "false")
 
 config :plausible, PlausibleWeb.Endpoint,
   url: [scheme: base_url.scheme, host: base_url.host, path: base_url.path, port: base_url.port],

lib/plausible_web/controllers/auth_controller.ex

@@ -25,7 +25,8 @@ defmodule PlausibleWeb.AuthController do
             ]
 
   def register_form(conn, _params) do
-    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) do
+    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) !=
+         "false" do
       redirect(conn, to: Routes.auth_path(conn, :login_form))
     else
       changeset = Plausible.Auth.User.changeset(%Plausible.Auth.User{})
@@ -38,7 +39,8 @@ defmodule PlausibleWeb.AuthController do
   end
 
   def register(conn, params) do
-    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) do
+    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) !=
+         "false" do
       redirect(conn, to: Routes.auth_path(conn, :login_form))
     else
       user = Plausible.Auth.User.new(params["user"])
@@ -74,7 +76,7 @@ defmodule PlausibleWeb.AuthController do
   end
 
   def register_from_invitation_form(conn, %{"invitation_id" => invitation_id}) do
-    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) do
+    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) == "true" do
       redirect(conn, to: Routes.auth_path(conn, :login_form))
     else
       invitation = Repo.get_by(Plausible.Auth.Invitation, invitation_id: invitation_id)
@@ -97,7 +99,7 @@ defmodule PlausibleWeb.AuthController do
   end
 
   def register_from_invitation(conn, %{"invitation_id" => invitation_id} = params) do
-    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) do
+    if Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) == "true" do
       redirect(conn, to: Routes.auth_path(conn, :login_form))
     else
       invitation = Repo.get_by(Plausible.Auth.Invitation, invitation_id: invitation_id)

lib/plausible_web/templates/auth/login_form.html.eex

@@ -16,7 +16,7 @@
     <p class="text-gray-500 text-xs my-2">Forgot password? <a href="/password/request-reset" class="underline text-gray-800 dark:text-gray-50">Click here</a> to reset it.</p>
   </div>
   <%= submit "Login →", class: "button mt-4 w-full" %>
-  <%= if !Keyword.fetch!(Application.get_env(:plausible, :selfhost),:disable_registration) do %>
+  <%= if Keyword.fetch!(Application.get_env(:plausible, :selfhost),:disable_registration) == "false" do %>
      <p class="text-center text-gray-500 text-xs mt-4">
       Don't have an account? <%= link("Register", to: "/register", class: "text-gray-800 dark:text-gray-50 underline") %> instead.
       </p>

lib/plausible_web/templates/layout/_header.html.eex

@@ -54,7 +54,7 @@
                     </div>
                 </li>
               </ul>
-            <%  Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) -> %>
+            <%  Keyword.fetch!(Application.get_env(:plausible, :selfhost), :disable_registration) != "false" -> %>
               <ul class="flex" x-show="!document.cookie.includes('logged_in=true')">
                 <li>
                     <div class="inline-flex">

test/plausible_web/controllers/admin_auth_controller_test.exs

@@ -30,7 +30,7 @@ defmodule PlausibleWeb.AdminAuthControllerTest do
     end
 
     test "disable registration", %{conn: conn} do
-      set_config(disable_registration: true)
+      set_config(disable_registration: "true")
       conn = get(conn, "/register")
       assert redirected_to(conn) == "/login"
     end
@@ -39,7 +39,7 @@ defmodule PlausibleWeb.AdminAuthControllerTest do
   def set_config(config) do
     updated_config =
       Keyword.merge(
-        [disable_authentication: false, disable_registration: false],
+        [disable_authentication: false, disable_registration: "false"],
         config
       )