mastodon websocket: use pattern match to get query data, robustly handle errors
authorWilliam Pitcock <nenolod@dereferenced.org>
Thu, 28 Feb 2019 16:02:48 +0000 (16:02 +0000)
committerWilliam Pitcock <nenolod@dereferenced.org>
Thu, 28 Feb 2019 16:02:48 +0000 (16:02 +0000)
lib/pleroma/web/mastodon_api/websocket_handler.ex

index f19f26b9ac91b6108e8b59734361e5c92f1b2c88..edab961f0a44faf5c80fb0252013498737541bb0 100644 (file)
@@ -23,9 +23,8 @@ defmodule Pleroma.Web.MastodonAPI.WebsocketHandler do
   ]
   @anonymous_streams ["public", "public:local", "hashtag"]
 
-  def init(req, _state) do
-    with {qs, req} <- :cowboy_req.qs(req),
-         params <- :cow_qs.parse_qs(qs),
+  def init(%{qs: qs} = req, _state) do
+    with params <- :cow_qs.parse_qs(qs),
          access_token <- List.keyfind(params, "access_token", 0),
          {_, stream} <- List.keyfind(params, "stream", 0),
          {:ok, user} <- allow_request(stream, access_token),
@@ -39,6 +38,7 @@ defmodule Pleroma.Web.MastodonAPI.WebsocketHandler do
 
       error ->
         Logger.debug("#{__MODULE__} denied connection: #{inspect(error)} - #{inspect(req)}")
+        {:ok, req} = :cowboy_req.reply(400, req)
         {:stop, req}
     end
   end