Thank you, we will look into it
-- Best Regards Mingyu Chen Email: [email protected] At 2026-02-04 00:00:14, "Ivan Orekhov" <[email protected]> wrote: >Hello Doris developers and committers, > >Thank you for your time and for maintaining the Doris project. > >I would like to kindly request a review for my pull request that adds >support for secure LDAP (LDAPS) connections to the Doris >authentication module. > >**PR Link:** https://github.com/apache/doris/pull/60275 > >**Summary of Changes:** >* Introduces a new configuration property `ldap_use_ssl` in >ldap.conf and LdapConfig.java. >* Modifies LdapConfig.java to construct the connection URL >(`ldap://` or `ldaps://`) based on this property and LdapClient.java >to use this method. >* Includes a new unit test in LdapClientTest.java to verify both the >default (LDAP) and new (LDAPS) behaviors. > >**Current Status:** >* The PR is marked as "ready for review". >* All standard CI checks have passed successfully (build, unit tests). >* The branch has no conflicts with the base branch (`master`). > >This feature is essential for environments with security policies that >mandate encrypted communication with LDAP servers. The change is >backward compatible, as the default behavior (LDAP) remains unchanged >unless the new property is explicitly set to `true`. > >I would be grateful if a committer or a developer familiar with the >authentication module could find time to review this contribution. I >am ready to provide any additional information or make necessary >adjustments based on the feedback. > >Best regards, >Ivan Orekhov
