Formatting fixes
authorEkaterina Vaartis <vaartis@cock.li>
Sat, 15 Dec 2018 19:08:26 +0000 (22:08 +0300)
committerEkaterina Vaartis <vaartis@cock.li>
Sat, 15 Dec 2018 19:08:26 +0000 (22:08 +0300)
config/config.exs
lib/pleroma/captcha/captcha_service.ex
lib/pleroma/captcha/kocaptcha.ex
lib/pleroma/web/twitter_api/twitter_api.ex

index 32593045c237d0c82daf08d9dd27ec8d97615410..b7d439e9da0f7c261ea37383c2bf832ff3fbc66d 100644 (file)
@@ -15,8 +15,7 @@ config :pleroma, Pleroma.Captcha,
   method: Pleroma.Captcha.Kocaptcha
 
 # Kocaptcha is a very simple captcha service, the source code is here: https://github.com/koto-bank/kocaptcha
-config :pleroma, Pleroma.Captcha.Kocaptcha,
-  endpoint: "http://localhost:9093"
+config :pleroma, Pleroma.Captcha.Kocaptcha, endpoint: "http://localhost:9093"
 
 # Upload configuration
 config :pleroma, Pleroma.Upload,
index ae1d6e7c715a0f71bc66eb8e31501ac3fc272784..907a73ad06477bdda81f40a2a3442f67b9ade15e 100644 (file)
@@ -1,5 +1,4 @@
 defmodule Pleroma.Captcha.Service do
-
   @doc """
   Request new captcha from a captcha service.
 
@@ -20,5 +19,5 @@ defmodule Pleroma.Captcha.Service do
 
   `true` if captcha is valid, `false` if not
   """
-  @callback validate(token :: String.t, captcha :: String.t) :: boolean
+  @callback validate(token :: String.t(), captcha :: String.t()) :: boolean
 end
index abccbf6d395886fb2584c2fd887b0da54723977e..173ce17f7c3e94422de08ffa747c6305991581e8 100644 (file)
@@ -7,9 +7,11 @@ defmodule Pleroma.Captcha.Kocaptcha do
   @impl Service
   def new() do
     endpoint = Pleroma.Config.get!([__MODULE__, :endpoint])
+
     case HTTPoison.get(endpoint <> "/new") do
       {:error, _} ->
         %{error: "Kocaptcha service unavailable"}
+
       {:ok, res} ->
         json_resp = Poison.decode!(res.body)
 
@@ -25,7 +27,7 @@ defmodule Pleroma.Captcha.Kocaptcha do
   def validate(token, captcha) do
     with false <- is_nil(captcha),
          [{^token, saved_md5}] <- :ets.lookup(@ets, token),
-         true <- (:crypto.hash(:md5, captcha) |> Base.encode16) == String.upcase(saved_md5) do
+         true <- :crypto.hash(:md5, captcha) |> Base.encode16() == String.upcase(saved_md5) do
       # Clear the saved value
       :ets.delete(@ets, token)
 
index 9f98c43c90cd3a56c96710db7a5415cce188a013..90b8345c5414639e06e3dc22fc8038c96d890a04 100644 (file)
@@ -139,11 +139,12 @@ defmodule Pleroma.Web.TwitterAPI.TwitterAPI do
 
     captcha_enabled = Pleroma.Config.get([Pleroma.Captcha, :enabled])
     # true if captcha is disabled or enabled and valid, false otherwise
-    captcha_ok = if !captcha_enabled do
-      true
-    else
-      Pleroma.Captcha.validate(params[:captcha_token], params[:captcha_solution])
-    end
+    captcha_ok =
+      if !captcha_enabled do
+        true
+      else
+        Pleroma.Captcha.validate(params[:captcha_token], params[:captcha_solution])
+      end
 
     # Captcha invalid
     if not captcha_ok do
@@ -155,8 +156,8 @@ defmodule Pleroma.Web.TwitterAPI.TwitterAPI do
       # no need to query DB if registration is open
       token =
         unless registrations_open || is_nil(tokenString) do
-        Repo.get_by(UserInviteToken, %{token: tokenString})
-      end
+          Repo.get_by(UserInviteToken, %{token: tokenString})
+        end
 
       cond do
         registrations_open || (!is_nil(token) && !token.used) ->
@@ -168,18 +169,17 @@ defmodule Pleroma.Web.TwitterAPI.TwitterAPI do
           else
             {:error, changeset} ->
               errors =
-              Ecto.Changeset.traverse_errors(changeset, fn {msg, _opts} -> msg end)
-              |> Jason.encode!()
+                Ecto.Changeset.traverse_errors(changeset, fn {msg, _opts} -> msg end)
+                |> Jason.encode!()
 
-            {:error, %{error: errors}}
+              {:error, %{error: errors}}
           end
 
-
         !registrations_open && is_nil(token) ->
-            {:error, "Invalid token"}
+          {:error, "Invalid token"}
 
         !registrations_open && token.used ->
-            {:error, "Expired token"}
+          {:error, "Expired token"}
       end
     end
   end