-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Vault UI Does not Configure LDAP Auth BindDN Correctly #6227
Comments
@eugene-davis thanks for adding this issue! I believe the UI is just passing through the value without editing it or escaping anything. Can you make up and share a fake password with special character in it? And then share how you provide it in the UI and CLI, both escaped and not escaped? Thanks! |
@tyrannosaurus-becks sure, let's use CLI - Unescaped and Non-Functional
I know for a fact this version returns the same authentication errors as it was the first way I attempted to perform configuration, but I don't have a wireshark capture so its possible it behaves differently than the UI issue. CLI - Escaped and Working
UI |
Fixed by #7206 |
Describe the bug
When using the UI to set up LDAP auth as opposed to the CLI, the BindDN account is not used, and instead it attempts to bind using the user's account. When it is configured via the CLI, it works as expected.
To Reproduce
Steps to reproduce the behavior:
Error authenticating: Error Making AP request
message (reproduced in full below).Expected behavior
Configuration from the UI and Vault CI should result in identical configurations.
Environment:
vault status
): 1.0.2vault version
): v1.0.2 ('37a1dc9c477c1c68c022d2084550f25bf20cac33')Vault server configuration file(s):
Additional context
The password for the bind user contained a special character, maybe the UI doesn't escape the string correctly? I had to escape it in the call the Vault CLI, but the same escape doesn't solve the problem in the web UI.
Flawed Capture Sample from LDAP (Wireshark "Follow Stream" scrubbed):
This is associated with this message from the CLI:
The text was updated successfully, but these errors were encountered: