Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

auth: set ldap base dn. Fix ldap docker example #1617

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Conversation

AndreyNenashev
Copy link
Contributor

while we can use a pattern DN to filter search results, we still need to specify a base DN to define the starting point for LDAP searches

@AndreyNenashev AndreyNenashev requested review from a team as code owners February 12, 2024 20:39
@github-actions github-actions bot added the kind: bug Something isn't working label Feb 12, 2024
Copy link

sonarcloud bot commented Feb 12, 2024

Quality Gate Passed Quality Gate passed for 'odd-platform-api'

Issues
0 New issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

Test Results

  55 files    55 suites   2m 38s ⏱️
259 tests 259 ✔️ 0 💤 0
287 runs  287 ✔️ 0 💤 0

Results for commit 29d1ec7.

@AndreyNenashev AndreyNenashev merged commit 4026a71 into main Feb 13, 2024
7 checks passed
@AndreyNenashev AndreyNenashev deleted the auth/ldap branch February 13, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants