-
Notifications
You must be signed in to change notification settings - Fork 417
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
Add ldap3 #971
Open
EBII
wants to merge
5
commits into
Kozea:master
Choose a base branch
from
EBII:addLdap3
base: master
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Add ldap3 #971
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…a distinguished name to bind with for connecting to the server. When this is successful user is authenticated. When an error occurs authentication is not successful. This relies on the ldap3 library which must be installed when using this feature. The feature can be configured with the config parameters 'ldap_server_uri' and 'ldap_bind_dn', when auth type is set to 'ldap'. Usage of 'ldap_server_uri' should be quite straight forward, 'ldap_bind_dn' is the distinguished name to bind with when connecting to the server. This may also include variables: %%u ... given login as a whole %%n ... local part, e.g. in 'user@example.com' only 'user' is used %%d ... domain part, e.g. in 'user@example.com' only 'example.com' is used here this is similar to dovecots variables as stated here: https://wiki.dovecot.org/Variables
need ldap3 python lib don't find requirements.txt or depends on other place it is for pr/Kozea#952
You should make it as a plugin like etc https://github.com/Unrud/RadicaleIMAP |
Unrud
force-pushed
the
master
branch
3 times, most recently
from
August 31, 2020 12:14
767d614
to
41bccb2
Compare
Potentially superseeded by #1218 |
Sophisticated user authentication should be handover'ed to reverse proxy, as they have a bunch of authentication options already build-in or available via plug-in modules. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
try to pass travis test to add ldap from pr/#952 the is set red cross due to don't have instal the ldap3 lib