public/mailtrain
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

ldap change to url for making it possible to use ldaps:// too

Browse source
This commit is contained in:
Adrian Woeltche 2018-06-21 15:15:44 +02:00
parent 206e7a5b52
commit 5325f2ea78
2 changed files with 4 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

6
config/default.toml
View file

@ -114,8 +114,7 @@ host="0.0.0.0"
[ldap]
# enable to use ldap user backend
enabled=false
host="localhost"
port=3002
url="ldap://localhost:3002"
baseDN="ou=users,dc=company"
filter="(|(username={{username}})(mail={{username}}))"
#Username field in LDAP (uid/cn/username)
@ -128,8 +127,7 @@ bindPassword=""
[ldapauth]
# Alternative LDAP implementation using the more popular passport-ldapauth library.
enabled=false
host="localhost"
port=389
url="ldap://localhost:389"
# Subtree in which the searchrequest for the user is done
baseDN="ou=users,dc=company"
# What whe are searching for. This should return a single user.

4
lib/passport.js
View file

@ -87,7 +87,7 @@ if (config.ldap.enabled && LdapStrategy) {
let opts = {
server: {
url: 'ldap://' + config.ldap.host + ':' + config.ldap.port
url: config.ldap.url
},
base: config.ldap.baseDN,
search: {
@ -130,7 +130,7 @@ if (config.ldap.enabled && LdapStrategy) {
log.info('Using LDAP auth (passport-ldapauth)');
let opts = {
server: {
url: 'ldap://' + config.ldap.host + ':' + config.ldap.port,
url: config.ldapauth.url,
searchBase: config.ldapauth.baseDN,
searchFilter: config.ldapauth.filter,
searchAttributes: [config.ldapauth.uidTag, 'mail'],