search

DerManoMann / ldap-auth-service-provider

A Silex Ldap authentication service provider
MIT License
5 stars 2 forks source link

Use dn for auth #17

Closed matracine closed 8 years ago

matracine commented 8 years ago

I propose to add an option 'useDN' to ldapAuthenticationProvider, so there is no BC break. If useDN is set to true, attribute dn has to be retrieved by ldapUserProvider.

matracine commented 8 years ago

Just had an idea, 'useDN' can be the name of the method to use to retreive the dn ('useDN' =>'getAuthName'). So no hard dependency is introduced between LdapUser and the authentication provider.

matracine commented 8 years ago

Hu ? I don't understand why you merged my pull request. Your solution was a lot better...