[enhance](auth) introduction of ldaps support via configuration property#60275
[enhance](auth) introduction of ldaps support via configuration property#60275morningman merged 12 commits intoapache:masterfrom
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
e6954e3 to
59a60c2
Compare
|
run buildall |
|
run feut |
1 similar comment
|
run feut |
2dc20f8 to
2cc92da
Compare
|
run feut |
1 similar comment
|
run feut |
4452270 to
c73cc70
Compare
|
run feut |
1 similar comment
|
run feut |
FE UT Coverage ReportIncrement line coverage |
|
run buildall |
TPC-H: Total hot run time: 33048 ms |
ClickBench: Total hot run time: 28.47 s |
FE Regression Coverage ReportIncrement line coverage |
|
run buildall |
|
run p0 |
TPC-H: Total hot run time: 32785 ms |
ClickBench: Total hot run time: 28.41 s |
FE Regression Coverage ReportIncrement line coverage |
1 similar comment
FE Regression Coverage ReportIncrement line coverage |
|
run cloud_p0 |
FE Regression Coverage ReportIncrement line coverage |
|
run p0 |
FE Regression Coverage ReportIncrement line coverage |
|
run buildall |
TPC-H: Total hot run time: 26827 ms |
TPC-DS: Total hot run time: 168894 ms |
FE UT Coverage ReportIncrement line coverage |
|
Hello, @morningman |
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
TPC-H: Total hot run time: 26837 ms |
TPC-DS: Total hot run time: 168007 ms |
|
Hello, @morningman |
…rty (#60275) ### What problem does this PR solve? This PR adds new configuration property **ldap_use_ssl** to enable usage of LDAPS to establish connection to LDAP instance. If **ldap_use_ssl** in ldap.conf is specified as **true** - LDAPS is used to create connection string. If **ldap_use_ssl** in ldap.conf is not specified or specified as **false** - LDAP is used to create connection string as now.
|
Hello, @morningman / @yiguolei ! |
@iaorekhov-1980 thank you so much, I just polished your document pr and it will be merged soon. |
…rty (#3351) This PR adds documentation for the LDAPS feature introduced in code PR #60275. Changes: Added documentation for the new ldap_use_ssl configuration property in ldap.conf. Added a note about configuring the Java trustStore for SSL certificate validation when LDAPS is enabled. Updated the configuration examples in the relevant documentation files. Documentation version coverage: The changes have been applied to mentioned directories to cover the requested branches: 1. docs/ (latest), 2. versioned_docs/version-3.1/, 3. versioned_docs/version-4.0/ Related PR: apache/doris#60275 Issue: close apache/doris#60236
What problem does this PR solve?
This PR adds new configuration property ldap_use_ssl to enable usage of LDAPS to establish connection to LDAP instance.
If ldap_use_ssl in ldap.conf is specified as true - LDAPS is used to create connection string.
If ldap_use_ssl in ldap.conf is not specified or specified as false - LDAP is used to create connection string as now.
Could you please include this PR into 4.x and 3.1.x branches, please!
Issue Number: close #60236
Related PR: #xxx
Problem Summary:
Currently it is not possible to use LDAPS to create connection to secured LDAP instances.
New configuration property allows to connect to such instances, but default behavior still relies on LDAP as is.
Release note
None
Check List (For Author)
Test
Behavior changed:
Check List (For Reviewer who merge this PR)