Remove authz

lib/beacon/beacon.ex

@@ -16,7 +16,6 @@ defmodule Beacon do
   * `Beacon.Lifecycle` - inject custom logic into Beacon lifecycle to change how pages are loaded an rendred, and more.
   * `Beacon.Content` - manage content as layouts, pages, page variants, snippets, and more.
   * `Beacon.MediaLibrary` - upload images, videos, and documents that can be used in your content.
-  * `Beacon.Authorization` - define permissions to limit access to content and features, also used on Beacon LiveAdmin.
 
   Get started with [your first site](https://hexdocs.pm/beacon/your-first-site.html) and check out the guides for more information.
 
@@ -53,8 +52,7 @@ defmodule Beacon do
       config :my_app, Beacon,
         sites: [
           [site: :my_site, endpoint: MyAppWeb.Endpoint]
-        ],
-        authorization_source: MyApp.AuthorizationPolicy
+        ]
 
       # lib/my_app/application.ex
       def start(_type, _args) do
@@ -87,22 +85,10 @@ defmodule Beacon do
 
     :pg.start_link(:beacon_cluster)
 
-    authorization_source = Keyword.get(opts, :authorization_source)
-
-    children =
-      sites
-      |> Enum.map(fn site_config -> assign_authorization_source(site_config, authorization_source) end)
-      |> Enum.map(&site_child_spec/1)
-
+    children = Enum.map(sites, &site_child_spec/1)
     Supervisor.init(children, strategy: :one_for_one)
   end
 
-  defp assign_authorization_source(site_config, nil), do: site_config
-
-  defp assign_authorization_source(site_config, authorization_source) do
-    Keyword.put_new(site_config, :authorization_source, authorization_source)
-  end
-
   defp site_child_spec(opts) do
     config = Config.new(opts)
     Supervisor.child_spec({Beacon.SiteSupervisor, config}, id: config.site)

lib/beacon/config.ex

@@ -55,11 +55,6 @@ defmodule Beacon.Config do
   """
   @type skip_boot? :: boolean()
 
-  @typedoc """
-  A module that implements `Beacon.Authorization.Policy`, used to provide authorization rules.
-  """
-  @type authorization_source :: module()
-
   @typedoc """
   A module that implements `Beacon.RuntimeCSS`.
   """
@@ -184,7 +179,6 @@ defmodule Beacon.Config do
           router: router(),
           repo: repo(),
           skip_boot?: skip_boot?(),
-          authorization_source: authorization_source(),
           css_compiler: css_compiler(),
           tailwind_config: tailwind_config(),
           live_socket_path: live_socket_path(),
@@ -218,7 +212,8 @@ defmodule Beacon.Config do
             router: nil,
             repo: nil,
             skip_boot?: false,
-            authorization_source: Beacon.Authorization.DefaultPolicy,
+            # TODO: rename to `authorization_policy`, see https://github.com/BeaconCMS/beacon/pull/563
+            # authorization_source: Beacon.Authorization.DefaultPolicy,
             css_compiler: Beacon.RuntimeCSS.TailwindCompiler,
             tailwind_config: nil,
             live_socket_path: "/live",
@@ -247,7 +242,6 @@ defmodule Beacon.Config do
           | {:router, router()}
           | {:repo, repo()}
           | {:skip_boot?, skip_boot?()}
-          | {:authorization_source, authorization_source()}
           | {:css_compiler, css_compiler()}
           | {:tailwind_config, tailwind_config()}
           | {:live_socket_path, live_socket_path()}
@@ -275,8 +269,6 @@ defmodule Beacon.Config do
 
     * `:skip_boot?` - `t:skip_boot?/0` (optional). Defaults to `false`.
 
-    * `:authorization_source` - `t:authorization_source/0` (optional). Defaults to `Beacon.Authorization.DefaultPolicy`.
-
     * `css_compiler` - `t:css_compiler/0` (optional). Defaults to `Beacon.RuntimeCSS.TailwindCompiler`.
 
     * `:tailwind_config` - `t:tailwind_config/0` (optional). Defaults to `Path.join(Application.app_dir(:beacon, "priv"), "tailwind.config.js")`.
@@ -313,7 +305,6 @@ defmodule Beacon.Config do
         endpoint: MyAppWeb.Endpoint,
         router: MyAppWeb.Router,
         repo: MyApp.Repo,
-        authorization_source: MyApp.MySiteAuthzPolicy,
         tailwind_config: Path.join(Application.app_dir(:my_app, "priv"), "tailwind.config.js"),
         template_formats: [
           {:custom_format, "My Custom Format"}
@@ -342,7 +333,6 @@ defmodule Beacon.Config do
         router: MyAppWeb.Router,
         repo: MyApp.Repo,
         skip_boot?: false,
-        authorization_source: MyApp.SiteAuthnPolicy,
         css_compiler: Beacon.RuntimeCSS.TailwindCompiler,
         tailwind_config: "/my_app/priv/tailwind.config.js",
         live_socket_path: "/live",

lib/errors.ex

@@ -34,18 +34,6 @@ defmodule Beacon.RuntimeError do
   defexception message: "runtime error in Beacon", plug_status: 404
 end
 
-defmodule Beacon.AuthorizationError do
-  @moduledoc """
-  Raised when there is an error in the `get_agent/1` or `authorized?/3` callbacks of
-  a `Beacon.Authorization.Policy`.
-
-  If you're seeing this error, and have implemented a custom `Beacon.Authorization.Policy`,
-  check the logic in your policy module.
-  """
-
-  defexception message: "error in Beacon.Authorization"
-end
-
 defmodule Beacon.ParserError do
   @moduledoc """
   Raised when Beacon's Markdown engine attempts to convert Markdown to HTML unsuccessfully.

mix.exs

@@ -174,11 +174,6 @@ defmodule Beacon.MixProject do
         Beacon.RuntimeCSS.TailwindCompiler,
         Beacon.Web.BeaconAssigns
       ],
-      "Authn and Authz": [
-        Beacon.Authorization,
-        Beacon.Authorization.Policy,
-        Beacon.Authorization.DefaultPolicy
-      ],
       Extensibility: [
         Beacon.Config,
         Beacon.Lifecycle,

test/beacon/config_test.exs

@@ -10,7 +10,6 @@ defmodule Beacon.ConfigTest do
     test "returns the site config" do
       assert %Beacon.Config{
                css_compiler: Beacon.RuntimeCSS.TailwindCompiler,
-               authorization_source: Beacon.BeaconTest.BeaconAuthorizationSource,
                live_socket_path: "/custom_live",
                safe_code_check: false,
                site: :my_site,

test/test_helper.exs

@@ -152,8 +152,7 @@ Supervisor.start_link(
            ]
          ]
        ]
-     ],
-     authorization_source: Beacon.BeaconTest.BeaconAuthorizationSource},
+     ]},
     Beacon.BeaconTest.Endpoint
   ],
   strategy: :one_for_one