Add is_admin flag, auto-grant to graham@mcintire.me
Adds a boolean is_admin column (default false) and has the registration changeset auto-set it to true when the email matches graham@mcintire.me (case-insensitive). The migration also backfills the flag for an existing row with that email.
This commit is contained in:
parent
0a0154137d
commit
ea5fdd6479
3 changed files with 53 additions and 0 deletions
|
|
@ -4,6 +4,8 @@ defmodule Microwaveprop.Accounts.User do
|
|||
|
||||
import Ecto.Changeset
|
||||
|
||||
@admin_email "graham@mcintire.me"
|
||||
|
||||
@primary_key {:id, :binary_id, autogenerate: true}
|
||||
@foreign_key_type :binary_id
|
||||
schema "users" do
|
||||
|
|
@ -13,11 +15,18 @@ defmodule Microwaveprop.Accounts.User do
|
|||
field :password, :string, virtual: true, redact: true
|
||||
field :hashed_password, :string, redact: true
|
||||
field :confirmed_at, :utc_datetime
|
||||
field :is_admin, :boolean, default: false
|
||||
field :authenticated_at, :utc_datetime, virtual: true
|
||||
|
||||
timestamps(type: :utc_datetime)
|
||||
end
|
||||
|
||||
@doc """
|
||||
Returns the email address that is automatically granted admin
|
||||
privileges on registration.
|
||||
"""
|
||||
def admin_email, do: @admin_email
|
||||
|
||||
@doc """
|
||||
A user changeset for registering a new user.
|
||||
|
||||
|
|
@ -40,6 +49,14 @@ defmodule Microwaveprop.Accounts.User do
|
|||
|> validate_email(opts)
|
||||
|> validate_confirmation(:password, message: "does not match password")
|
||||
|> validate_password(opts)
|
||||
|> maybe_grant_admin()
|
||||
end
|
||||
|
||||
defp maybe_grant_admin(changeset) do
|
||||
case get_field(changeset, :email) do
|
||||
nil -> changeset
|
||||
email -> if String.downcase(email) == @admin_email, do: put_change(changeset, :is_admin, true), else: changeset
|
||||
end
|
||||
end
|
||||
|
||||
defp validate_callsign(changeset, opts) do
|
||||
|
|
|
|||
|
|
@ -0,0 +1,17 @@
|
|||
defmodule Microwaveprop.Repo.Migrations.AddIsAdminToUsers do
|
||||
use Ecto.Migration
|
||||
|
||||
def up do
|
||||
alter table(:users) do
|
||||
add :is_admin, :boolean, null: false, default: false
|
||||
end
|
||||
|
||||
execute "UPDATE users SET is_admin = true WHERE email = 'graham@mcintire.me'"
|
||||
end
|
||||
|
||||
def down do
|
||||
alter table(:users) do
|
||||
remove :is_admin
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
@ -133,6 +133,25 @@ defmodule Microwaveprop.AccountsTest do
|
|||
assert is_binary(user.hashed_password)
|
||||
assert is_nil(user.password)
|
||||
assert is_nil(user.confirmed_at)
|
||||
refute user.is_admin
|
||||
end
|
||||
|
||||
test "grants is_admin to the configured admin email" do
|
||||
{:ok, user} =
|
||||
Accounts.register_user(valid_user_attributes(email: User.admin_email()))
|
||||
|
||||
assert user.is_admin
|
||||
end
|
||||
|
||||
test "grants is_admin regardless of email case" do
|
||||
upper = String.upcase(User.admin_email())
|
||||
{:ok, user} = Accounts.register_user(valid_user_attributes(email: upper))
|
||||
assert user.is_admin
|
||||
end
|
||||
|
||||
test "does not grant is_admin to other emails" do
|
||||
{:ok, user} = Accounts.register_user(valid_user_attributes())
|
||||
refute user.is_admin
|
||||
end
|
||||
end
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue