Fix LDAP user registration (#229)
authornullobsi <me@nullob.si>
Tue, 1 Nov 2022 14:17:55 +0000 (14:17 +0000)
committerfloatingghost <hannah@coffee-and-dreams.uk>
Tue, 1 Nov 2022 14:17:55 +0000 (14:17 +0000)
Simple fix for LDAP user registration. I'm not sure what changed but I managed to get Akkoma running in a debug session and figured out it was missing a match for an extra value at the end. I don't know Elixir all that well so I'm not sure if this was the correct way to do it... but it works. :)

Reviewed-on: https://akkoma.dev/AkkomaGang/akkoma/pulls/229
Co-authored-by: nullobsi <me@nullob.si>
Co-committed-by: nullobsi <me@nullob.si>
lib/pleroma/web/auth/ldap_authenticator.ex

index f77e8d20342648c012926639435b1e1b793dab8b..ccf4b41041b3f1c021a8fdd3747f3d761bf54f4b 100644 (file)
@@ -102,7 +102,7 @@ defmodule Pleroma.Web.Auth.LDAPAuthenticator do
            {:scope, :eldap.wholeSubtree()},
            {:timeout, @search_timeout}
          ]) do
-      {:ok, {:eldap_search_result, [{:eldap_entry, _, attributes}], _}} ->
+      {:ok, {:eldap_search_result, [{:eldap_entry, _, attributes}], _, _}} ->
         params = %{
           name: name,
           nickname: name,