[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 |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
89ad517 to
c4c0070
Compare
|
run feut |
|
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 ! |
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)