make Pleroma.Endpoint use extra_cookie_attrs in config
authorHakurei Reimu <admin@marchgenso.me>
Mon, 15 Apr 2019 04:33:46 +0000 (12:33 +0800)
committerHakurei Reimu <admin@marchgenso.me>
Tue, 16 Apr 2019 14:20:56 +0000 (22:20 +0800)
lib/pleroma/web/endpoint.ex

index 1633477c365dd8aa2649b515d6a55d3a507ecc08..7f939991d970a080ff98075a43e38c9a92d2bfc2 100644 (file)
@@ -58,14 +58,9 @@ defmodule Pleroma.Web.Endpoint do
       do: "__Host-pleroma_key",
       else: "pleroma_key"
 
-  same_site =
-    if Pleroma.Config.oauth_consumer_enabled?() do
-      # Note: "SameSite=Strict" prevents sign in with external OAuth provider
-      #   (there would be no cookies during callback request from OAuth provider)
-      "SameSite=Lax"
-    else
-      "SameSite=Strict"
-    end
+  extra =
+    Pleroma.Config.get([__MODULE__, :extra_cookie_attrs])
+    |> Enum.join(";")
 
   # The session will be stored in the cookie and signed,
   # this means its contents can be read but not tampered with.
@@ -77,7 +72,7 @@ defmodule Pleroma.Web.Endpoint do
     signing_salt: {Pleroma.Config, :get, [[__MODULE__, :signing_salt], "CqaoopA2"]},
     http_only: true,
     secure: secure_cookies,
-    extra: same_site
+    extra: extra
   )
 
   # Note: the plug and its configuration is compile-time this can't be upstreamed yet