Fix buggy condition in account validity handler (#7074)

This commit is contained in:
Brendan Abolivier 2020-03-16 12:16:30 +00:00 committed by GitHub
parent d8d91983bc
commit beb19cf61a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 1 deletions

1
changelog.d/7074.bugfix Normal file
View File

@ -0,0 +1 @@
Fix a bug causing account validity renewal emails to be sent even if the feature is turned off in some cases.

View File

@ -44,7 +44,11 @@ class AccountValidityHandler(object):
self._account_validity = self.hs.config.account_validity self._account_validity = self.hs.config.account_validity
if self._account_validity.renew_by_email_enabled and load_jinja2_templates: if (
self._account_validity.enabled
and self._account_validity.renew_by_email_enabled
and load_jinja2_templates
):
# Don't do email-specific configuration if renewal by email is disabled. # Don't do email-specific configuration if renewal by email is disabled.
try: try:
app_name = self.hs.config.email_app_name app_name = self.hs.config.email_app_name