search

Banno / getsentry-ldap-auth

A Sentry extension to add an LDAP server as an authention source.
Apache License 2.0
163 stars 53 forks source link

Update backend.py #17

Closed kmlebedev closed 7 years ago

kmlebedev commented 7 years ago

active directory compatible if filed mail not exist

barronhagerman commented 7 years ago

Sorry for the delay. I don't do much with Sentry Server these days.

I don't have a way to test with AD, but it still works in my configuration and the changes seem good to me. I haven't seen an AD user without a mail attribute, since that's a standard inetOrgPerson attribute, but I'm fine with this change.