Skip to content

KNOX-3422: Ldap Proxy used in-memory bind user - #1357

Merged
hanicz merged 2 commits into
apache:masterfrom
handavid:knox-3422-binduser
Aug 28, 2026
Merged

KNOX-3422: Ldap Proxy used in-memory bind user#1357
hanicz merged 2 commits into
apache:masterfrom
handavid:knox-3422-binduser

Conversation

@handavid

Copy link
Copy Markdown
Contributor

KNOX-3422 - Create LDAP Proxy bind user in-memory instead of in embedded LDAP

What changes were proposed in this pull request?

The bind user created from the gateway configuration is now created in-memory instead of being added to the embedded ldap server. This provides the benefit of being able to rotate the user just by changing the configuration. Previously, since the configured user is not tracked, there wasn't a way to automatically rotate the user.

How was this patch tested?

Unit test were updated and run.

Integration Tests

LDAP proxy search workflow tests were updated to use the configured bind user instead of a user in the remote LDAP.
A test was added to verify that anonymous bind does not work when the bind user is configured.

UI changes

no ui changes

The bind user created from the gateway configuration is now created
in-memory instead of being added to the embedded ldap server. This
provides the benefit of being able to rotate the user just by changing
the configuration. Previously, since the configured user is not tracked,
there wasn't a way to automatically rotate the user.
@handavid

Copy link
Copy Markdown
Contributor Author

@smolnar82 @hanicz

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Test Results

 3 files   3 suites   8s ⏱️
53 tests 53 ✅ 0 💤 0 ❌
64 runs  64 ✅ 0 💤 0 ❌

Results for commit 8f3c1a9.

♻️ This comment has been updated with latest results.

@handavid handavid left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hanicz I addressed you comments. thanks!

@hanicz
hanicz merged commit c776132 into apache:master Aug 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants