Skip to content

Commit b534f58

Browse files
Merge pull request #4288 from rabbitmq/mergify/bp/v3.8.x/pr-4287
LDAP: match on tuples instead of the #eldap_search_result record for OTP 24.3 compat (backport #4285) (backport #4286) (backport #4287)
2 parents c61671e + 88e5c02 commit b534f58

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

deps/rabbitmq_auth_backend_ldap/src/rabbit_auth_backend_ldap.erl

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -829,7 +829,12 @@ dn_lookup(Username, LDAP) ->
829829
{attributes, ["distinguishedName"]}]) of
830830
{ok, {referral, Referrals}} ->
831831
{error, {referrals_not_supported, Referrals}};
832-
{ok, #eldap_search_result{entries = [#eldap_entry{object_name = DN}]}}->
832+
%% support #eldap_search_result before and after
833+
%% https://github.com/erlang/otp/pull/5538
834+
{ok, {eldap_search_result, [#eldap_entry{object_name = DN}], _Referrals}}->
835+
?L1("DN lookup: ~s -> ~s", [Username, DN]),
836+
DN;
837+
{ok, {eldap_search_result, [#eldap_entry{object_name = DN}], _Referrals, _Controls}}->
833838
?L1("DN lookup: ~s -> ~s", [Username, DN]),
834839
DN;
835840
{ok, #eldap_search_result{entries = Entries}} ->

0 commit comments

Comments
 (0)