kyuubi icon indicating copy to clipboard operation
kyuubi copied to clipboard

[KYUUBI #3359] Minor change LdapAuthenticationProviderImpl to adapt AD

Open jetliu1711 opened this issue 3 years ago • 2 comments

Compatible with the Microsoft Active Directory LDAP. Currently, Apache Kyuubi support open LDAP, cannot support Microsoft Active Directory LDAP,I try much more times, So I try to debug LdapAuthenticationProviderImpl class,I find something wrong with use Active Directory LDAP to connect to kyuubi authorized.it can be connect ,but close connect have something wrong,so I clear the close method,it can be very well。support Active Directory LDAP。

Why are the changes needed?

How was this patch tested?

  • [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • [ ] Add screenshots for manual tests if appropriate

  • [ ] Run test locally before make a pull request

jetliu1711 avatar Aug 26 '22 10:08 jetliu1711

... but close connect have something wrong ...

Could please provide more details about the error? e.g. the exception stacktrace

pan3793 avatar Aug 26 '22 10:08 pan3793

#4152 fully ported LDAP-related functionalities from Apache Hive, it supposes to support Active Directory, would you please help to verify it?

pan3793 avatar Jan 31 '23 05:01 pan3793