Issue #1306 - Allow custom name attribute in LDAP - #1317

Thanks for this @dfanara. All looked and worked great! Merged ready for the next patch release.

Hey, @dfanara I had some issues while using this.

When I set the config option to displayName, nothing was being populated while if I set the config to displayname it worked. Our LDAP server is a Microsoft AD and if I do an ldapsearch from the CLI, the attribute is name with a capital N, while the code in BookStack changes that attribute to lowercase somewhere. This should either be noted in documentation or looked at (cc @ssddanbrown).

@craftbyte I've looked into your issue and I found that php-ldap converts all of the ldap attributes to lower case (http://php.net/manual/en/function.ldap-get-entries.php).

I'll submit a PR for a fix for this, but until it is released I think @ssddanbrown could add a note to the documentation.

Sorry for the confusion and thanks for pointing it out!