search

ccovey / ldap-auth

ldap-auth
69 stars 43 forks source link

AD data merging incorrectly into a local user. #9

Closed trq closed 11 years ago

trq commented 11 years ago

Just a reminder about our conversation on irc. It appears this line: https://github.com/ccovey/ldap-auth/blob/master/src/Ccovey/LdapAuth/LdapAuthUserProvider.php#L155 merges the local model with the AD data with the local data taking priority. I (we) believe this is the wrong way around.

ccovey commented 11 years ago

@trq this is fixed in master, when I get the other issues figured out i'll cut a new tag to be used.