Banno / getsentry-ldap-auth

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

add default value for AUTH_LDAP_SENTRY_GROUP_ROLE_MAPPING #42

Closed rodgomes closed 4 years ago

rodgomes commented 4 years ago

Not having a default value for this settings break sentry installation for application not expecting to have this, especially if they rely on semver.

aleksihakli commented 4 years ago

Nice catch @rodgomes! @barronhagerman could this be merged and released as e.g. 2.8.1? This broke the installation for us in recently.