diff --git a/config/config.exs b/config/config.exs index cb708b25e..22e512fe4 100644 --- a/config/config.exs +++ b/config/config.exs @@ -12,7 +12,7 @@ config :mobilizon, config :mobilizon, :instance, name: System.get_env("MOBILIZON_INSTANCE_NAME") || "Localhost", version: "1.0.0-dev", - registrations_open: true + registrations_open: System.get_env("MOBILIZON_INSTANCE_REGISTRATIONS_OPEN") || false config :mime, :types, %{ "application/activity+json" => ["activity-json"], diff --git a/config/test.exs b/config/test.exs index 3339f23a3..1ace33015 100644 --- a/config/test.exs +++ b/config/test.exs @@ -1,5 +1,9 @@ use Mix.Config +config :mobilizon, :instance, + name: "Test instance", + registrations_open: true + # We don't run a server during test. If one is required, # you can enable the server option below. config :mobilizon, MobilizonWeb.Endpoint, diff --git a/js/src/components/NavBar.vue b/js/src/components/NavBar.vue index 4eac95233..855ea2eea 100644 --- a/js/src/components/NavBar.vue +++ b/js/src/components/NavBar.vue @@ -18,7 +18,7 @@
- + Sign up @@ -51,6 +51,8 @@ import { deleteUserData } from '@/utils/auth'; import { LOGGED_PERSON } from '@/graphql/actor'; import { IActor, IPerson } from '@/types/actor.model'; import { RouteName } from '@/router'; +import { CONFIG } from '@/graphql/config'; +import { IConfig } from '@/types/config.model'; @Component({ apollo: { @@ -71,7 +73,10 @@ import { RouteName } from '@/router'; loggedPerson: { query: LOGGED_PERSON, }, - }, + config: { + query: CONFIG, + } + } }) export default class NavBar extends Vue { notifications = [ @@ -83,6 +88,7 @@ export default class NavBar extends Vue { searchText: string | null = null; searchSelect = null; loggedPerson!: IPerson; + config!: IConfig; get items() { return this.search.map(searchEntry => { diff --git a/js/src/graphql/config.ts b/js/src/graphql/config.ts new file mode 100644 index 000000000..46a73409f --- /dev/null +++ b/js/src/graphql/config.ts @@ -0,0 +1,10 @@ +import gql from 'graphql-tag'; + +export const CONFIG = gql` +query { + config { + name, + registrationsOpen + } +} +`; diff --git a/js/src/types/config.model.ts b/js/src/types/config.model.ts new file mode 100644 index 000000000..78ff7d284 --- /dev/null +++ b/js/src/types/config.model.ts @@ -0,0 +1,5 @@ +export interface IConfig { + name: string, + + registrationsOpen: boolean, +} diff --git a/js/src/views/User/Login.vue b/js/src/views/User/Login.vue index 770649bda..196ade974 100644 --- a/js/src/views/User/Login.vue +++ b/js/src/views/User/Login.vue @@ -37,7 +37,7 @@ Forgot your password ?
-
+
get_in([:registrations_open]) + |> to_bool + end + + def instance_name() do + instance_config() + |> get_in([:name]) + end + + defp instance_config(), do: Application.get_env(:mobilizon, :instance) + + defp to_bool(v), do: v == true or v == "true" or v == "True" +end diff --git a/lib/mobilizon/users/user.ex b/lib/mobilizon/users/user.ex index d1bdae752..cd31a29f4 100644 --- a/lib/mobilizon/users/user.ex +++ b/lib/mobilizon/users/user.ex @@ -146,13 +146,8 @@ defmodule Mobilizon.Users.User do end end - def is_confirmed(%User{confirmed_at: nil} = _user) do - {:error, :unconfirmed} - end - - def is_confirmed(%User{} = user) do - {:ok, user} - end + def is_confirmed(%User{confirmed_at: nil} = _user), do: {:error, :unconfirmed} + def is_confirmed(%User{} = user), do: {:ok, user} def owns_actor(%User{actors: actors}, actor_id) do case Enum.find(actors, fn a -> a.id == actor_id end) do diff --git a/lib/mobilizon_web/resolvers/config.ex b/lib/mobilizon_web/resolvers/config.ex new file mode 100644 index 000000000..f9b7b5569 --- /dev/null +++ b/lib/mobilizon_web/resolvers/config.ex @@ -0,0 +1,14 @@ +defmodule MobilizonWeb.Resolvers.Config do + @moduledoc """ + Handles the config-related GraphQL calls + """ + + import Mobilizon.CommonConfig + + @doc """ + Get config + """ + def get_config(_parent, _params, _context) do + {:ok, %{name: instance_name(), registrations_open: registrations_open?()}} + end +end diff --git a/lib/mobilizon_web/resolvers/user.ex b/lib/mobilizon_web/resolvers/user.ex index 974568196..fed884d19 100644 --- a/lib/mobilizon_web/resolvers/user.ex +++ b/lib/mobilizon_web/resolvers/user.ex @@ -3,6 +3,7 @@ defmodule MobilizonWeb.Resolvers.User do Handles the user-related GraphQL calls """ alias Mobilizon.Actors.Actor + alias Mobilizon.CommonConfig alias Mobilizon.Users.User alias Mobilizon.{Actors, Users} alias Mobilizon.Service.Users.{ResetPassword, Activation} @@ -64,15 +65,23 @@ defmodule MobilizonWeb.Resolvers.User do end @doc """ - Register an user : + Register an user: + - check registrations are enabled - create the user - send a validation email to the user """ @spec create_user(any(), map(), any()) :: tuple() def create_user(_parent, args, _resolution) do - with {:ok, %User{} = user} <- Users.register(args) do + with {:registrations_open, true} <- {:registrations_open, CommonConfig.registrations_open?()}, + {:ok, %User{} = user} <- Users.register(args) do Activation.send_confirmation_email(user) {:ok, user} + else + {:registrations_open, false} -> + {:error, "Registrations are not enabled"} + + err -> + err end end diff --git a/lib/mobilizon_web/schema.ex b/lib/mobilizon_web/schema.ex index 0373b80ce..4a8f868cf 100644 --- a/lib/mobilizon_web/schema.ex +++ b/lib/mobilizon_web/schema.ex @@ -18,6 +18,7 @@ defmodule MobilizonWeb.Schema do import_types(MobilizonWeb.Schema.Actors.PersonType) import_types(MobilizonWeb.Schema.Actors.GroupType) import_types(MobilizonWeb.Schema.CommentType) + import_types(MobilizonWeb.Schema.ConfigType) alias MobilizonWeb.Resolvers @@ -133,6 +134,7 @@ defmodule MobilizonWeb.Schema do import_fields(:participant_queries) import_fields(:tag_queries) import_fields(:address_queries) + import_fields(:config_queries) end @desc """ diff --git a/lib/mobilizon_web/schema/config.ex b/lib/mobilizon_web/schema/config.ex new file mode 100644 index 000000000..ea7e9259e --- /dev/null +++ b/lib/mobilizon_web/schema/config.ex @@ -0,0 +1,23 @@ +defmodule MobilizonWeb.Schema.ConfigType do + @moduledoc """ + Schema representation for User + """ + use Absinthe.Schema.Notation + + alias MobilizonWeb.Resolvers.Config + + @desc "A config object" + object :config do + # Instance name + field(:name, :string) + + field(:registrations_open, :boolean) + end + + object :config_queries do + @desc "Get the instance config" + field :config, :config do + resolve(&Config.get_config/3) + end + end +end diff --git a/test/mobilizon_web/resolvers/config_resolver_test.exs b/test/mobilizon_web/resolvers/config_resolver_test.exs new file mode 100644 index 000000000..d9cb1bc97 --- /dev/null +++ b/test/mobilizon_web/resolvers/config_resolver_test.exs @@ -0,0 +1,25 @@ +defmodule MobilizonWeb.Resolvers.ConfigResolverTest do + alias MobilizonWeb.AbsintheHelpers + use MobilizonWeb.ConnCase + use Bamboo.Test + + describe "Resolver: Get config" do + test "get_config/3 returns the instance config", context do + query = """ + { + config { + name, + registrationsOpen + } + } + """ + + res = + context.conn + |> get("/api", AbsintheHelpers.query_skeleton(query, "config")) + + assert json_response(res, 200)["data"]["config"]["name"] == "Test instance" + assert json_response(res, 200)["data"]["config"]["registrationsOpen"] == true + end + end +end diff --git a/test/mobilizon_web/resolvers/user_resolver_test.exs b/test/mobilizon_web/resolvers/user_resolver_test.exs index ffd47e4cd..5db02f088 100644 --- a/test/mobilizon_web/resolvers/user_resolver_test.exs +++ b/test/mobilizon_web/resolvers/user_resolver_test.exs @@ -1,11 +1,12 @@ defmodule MobilizonWeb.Resolvers.UserResolverTest do use MobilizonWeb.ConnCase - alias Mobilizon.{Actors, Users} + alias Mobilizon.{Actors, Users, CommonConfig} alias Mobilizon.Actors.Actor alias Mobilizon.Users.User alias MobilizonWeb.AbsintheHelpers alias Mobilizon.Service.Users.ResetPassword import Mobilizon.Factory + import Mock use Bamboo.Test @valid_actor_params %{email: "test@test.tld", password: "testest", username: "test"} @@ -389,6 +390,29 @@ defmodule MobilizonWeb.Resolvers.UserResolverTest do assert hd(json_response(res, 200)["errors"])["message"] == "Email doesn't fit required format" end + + test "test create_user/3 doesn't create a user when registration is disabled", context do + with_mock CommonConfig, registrations_open?: fn -> false end do + mutation = """ + mutation { + createUser( + email: "#{@user_creation.email}", + password: "#{@user_creation.password}", + ) { + id, + email + } + } + """ + + res = + context.conn + |> post("/api", AbsintheHelpers.mutation_skeleton(mutation)) + + assert hd(json_response(res, 200)["errors"])["message"] == + "Registrations are not enabled" + end + end end describe "Resolver: Validate an user" do