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

Add ldap3 #971

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add ldap3 #971

wants to merge 5 commits into from

Conversation

EBII
Copy link

@EBII EBII commented Jul 18, 2019

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

marco74 and others added 5 commits May 11, 2019 16:04
…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
@Tntdruid
Copy link

You should make it as a plugin like etc https://github.com/Unrud/RadicaleIMAP

@pbiering
Copy link
Collaborator

Potentially superseeded by #1218

@pbiering
Copy link
Collaborator

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants