MastoAPI streaming: Replace access_token with Sec-WebSocket-Protocol
authorKokaKiwi <kokakiwi@kokakiwi.net>
Sat, 22 Jun 2019 17:45:21 +0000 (19:45 +0200)
committerHaelwenn (lanodan) Monnier <contact@hacktivis.me>
Sat, 6 Jul 2019 18:29:17 +0000 (20:29 +0200)
lib/pleroma/web/mastodon_api/websocket_handler.ex

index 3299e1721948a754c3fba44dd4d4863aa5be3ae3..db6ae23b05264d68e1339c1d39a0f5bb6a8cfa23 100644 (file)
@@ -29,7 +29,7 @@ defmodule Pleroma.Web.MastodonAPI.WebsocketHandler do
 
   def init(%{qs: qs} = req, state) do
     with params <- :cow_qs.parse_qs(qs),
-         access_token <- List.keyfind(params, "access_token", 0),
+         access_token <- :cowboy_req.header("sec-websocket-protocol", req, 0),
          {_, stream} <- List.keyfind(params, "stream", 0),
          {:ok, user} <- allow_request(stream, access_token),
          topic when is_binary(topic) <- expand_topic(stream, params) do
@@ -89,7 +89,7 @@ defmodule Pleroma.Web.MastodonAPI.WebsocketHandler do
   end
 
   # Authenticated streams.
-  defp allow_request(stream, {"access_token", access_token}) when stream in @streams do
+  defp allow_request(stream, access_token) when stream in @streams do
     with %Token{user_id: user_id} <- Repo.get_by(Token, token: access_token),
          user = %User{} <- User.get_cached_by_id(user_id) do
       {:ok, user}