diff options
author | Mark Felder <feld@feld.me> | 2023-05-29 13:59:51 -0400 |
---|---|---|
committer | Mark Felder <feld@feld.me> | 2023-05-29 13:59:51 -0400 |
commit | 843fcca5b4d022e4c088d4a60839b4a286500148 (patch) | |
tree | 95acf70716457a7098537324edf008afb18a1099 /lib | |
parent | 506a1c98e716754455387be9ace3ad7aec9c47a3 (diff) | |
download | pleroma-843fcca5b4d022e4c088d4a60839b4a286500148.tar.gz pleroma-843fcca5b4d022e4c088d4a60839b4a286500148.zip |
Validate Host header matches expected value before allowing access to MediaProxy
Diffstat (limited to 'lib')
-rw-r--r-- | lib/pleroma/web/media_proxy/media_proxy_controller.ex | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/lib/pleroma/web/media_proxy/media_proxy_controller.ex b/lib/pleroma/web/media_proxy/media_proxy_controller.ex index bda5b36ed..767496b68 100644 --- a/lib/pleroma/web/media_proxy/media_proxy_controller.ex +++ b/lib/pleroma/web/media_proxy/media_proxy_controller.ex @@ -12,6 +12,7 @@ defmodule Pleroma.Web.MediaProxy.MediaProxyController do alias Pleroma.Web.MediaProxy alias Plug.Conn + plug(:validate_host) plug(:sandbox) def remote(conn, %{"sig" => sig64, "url" => url64}) do @@ -205,6 +206,17 @@ defmodule Pleroma.Web.MediaProxy.MediaProxyController do Config.get([:media_proxy, :proxy_opts], []) end + defp validate_host(conn, _params) do + proxy_host = MediaProxy.base_url() |> URI.parse() |> Map.get(:host) + + if match?(^proxy_host, conn.host) do + conn + else + send_resp(conn, 400, Conn.Status.reason_phrase(400)) + |> halt() + end + end + defp sandbox(conn, _params) do conn |> merge_resp_headers([{"content-security-policy", "sandbox;"}]) |