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

Clarify documentation on intent of application #127

Closed
Fmstrat opened this issue Mar 30, 2023 · 2 comments
Closed

Clarify documentation on intent of application #127

Fmstrat opened this issue Mar 30, 2023 · 2 comments

Comments

@Fmstrat
Copy link

Fmstrat commented Mar 30, 2023

Since there is nothing in the README that states otherwise, I and I'm seeing others (example: #5) are easily confused by the purpose of this connector, thinking that it synchronizes users via LDAP.

A version of the statement from the Wiki should be included at the very top of this README:

This service does not synchronize users between your LDAP server and Vaultwarden. Instead, LDAP integration is performed by querying your LDAP server and inviting users to your Vaultwarden instance.

@ViViDboarder
Copy link
Owner

Sure, I do state this as the second line in the eadme:

After configuring, run vaultwarden_ldap and it will invite any users it finds in LDAP to your vaultwarden instance.

However, I do see that is ambiguous as it doesn't explicitly state that it is all it does. I'll make some edits.

@Fmstrat
Copy link
Author

Fmstrat commented Mar 30, 2023

Thanks!

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

No branches or pull requests

2 participants