From 1b98cd86104bc8ffbbb550a6770deb94b5dbbfc7 Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Fri, 8 Jan 2021 07:47:55 -0600
Subject: [PATCH] Remove ConfirmUserPlug

---
 lib/pleroma/web/plugs/confirm_user_plug.ex    | 30 -------------------
 lib/pleroma/web/router.ex                     |  1 -
 .../web/plugs/confirm_user_plug_test.exs      | 30 -------------------
 3 files changed, 61 deletions(-)
 delete mode 100644 lib/pleroma/web/plugs/confirm_user_plug.ex
 delete mode 100644 test/pleroma/web/plugs/confirm_user_plug_test.exs

diff --git a/lib/pleroma/web/plugs/confirm_user_plug.ex b/lib/pleroma/web/plugs/confirm_user_plug.ex
deleted file mode 100644
index 218068de0..000000000
--- a/lib/pleroma/web/plugs/confirm_user_plug.ex
+++ /dev/null
@@ -1,30 +0,0 @@
-# Pleroma: A lightweight social networking server
-# Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/>
-# SPDX-License-Identifier: AGPL-3.0-only
-
-defmodule Pleroma.Web.Plugs.ConfirmUserPlug do
-  @moduledoc """
-  If a user has ever been granted an OAuth token, they are eligible to become
-  confirmed, regardless of the account_activation_required setting. This plug
-  will confirm a user if found.
-  """
-
-  alias Pleroma.User
-  import Plug.Conn
-
-  def init(opts), do: opts
-
-  def call(%{assigns: %{user: %User{confirmation_pending: true} = user}} = conn, _opts) do
-    with {:ok, user} <- confirm_user(user) do
-      assign(conn, :user, user)
-    end
-  end
-
-  def call(conn, _opts), do: conn
-
-  defp confirm_user(%User{} = user) do
-    user
-    |> User.confirmation_changeset(need_confirmation: false)
-    |> User.update_and_set_cache()
-  end
-end
diff --git a/lib/pleroma/web/router.ex b/lib/pleroma/web/router.ex
index 0deb64735..aefc9f0be 100644
--- a/lib/pleroma/web/router.ex
+++ b/lib/pleroma/web/router.ex
@@ -56,7 +56,6 @@ defmodule Pleroma.Web.Router do
     plug(Pleroma.Web.Plugs.UserEnabledPlug)
     plug(Pleroma.Web.Plugs.SetUserSessionIdPlug)
     plug(Pleroma.Web.Plugs.EnsureUserTokenAssignsPlug)
-    plug(Pleroma.Web.Plugs.ConfirmUserPlug)
   end
 
   pipeline :base_api do
diff --git a/test/pleroma/web/plugs/confirm_user_plug_test.exs b/test/pleroma/web/plugs/confirm_user_plug_test.exs
deleted file mode 100644
index 43c1c28a9..000000000
--- a/test/pleroma/web/plugs/confirm_user_plug_test.exs
+++ /dev/null
@@ -1,30 +0,0 @@
-# Pleroma: A lightweight social networking server
-# Copyright © 2017-2020 Pleroma Authors <https://pleroma.social/>
-# SPDX-License-Identifier: AGPL-3.0-only
-
-defmodule Pleroma.Web.Plugs.ConfirmUserPlugTest do
-  use Pleroma.Web.ConnCase, async: true
-  alias Pleroma.User
-  alias Pleroma.Web.Plugs.ConfirmUserPlug
-  import Pleroma.Factory
-
-  test "it confirms an unconfirmed user", %{conn: conn} do
-    %User{id: user_id} = user = insert(:user, confirmation_pending: true)
-
-    conn =
-      conn
-      |> assign(:user, user)
-      |> ConfirmUserPlug.call(%{})
-
-    assert %Plug.Conn{assigns: %{user: %User{id: ^user_id, confirmation_pending: false}}} = conn
-    assert %User{confirmation_pending: false} = User.get_by_id(user_id)
-  end
-
-  test "it does nothing without an unconfirmed user", %{conn: conn} do
-    assert conn == ConfirmUserPlug.call(conn, %{})
-
-    user = insert(:user, confirmation_pending: false)
-    conn = assign(conn, :user, user)
-    assert conn == ConfirmUserPlug.call(conn, %{})
-  end
-end