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

Reply via email to