New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(2fa): Send 'security settings changed' email with 2fa enroll #8771
Conversation
@@ -195,7 +195,7 @@ def post(self, request, user, interface_id): | |||
context={ | |||
'authenticator': interface.authenticator, | |||
}, | |||
send_email=False, | |||
send_email=True, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔 I wonder why we didn't do this to begin with.
@@ -203,7 +214,10 @@ def test_recovery_codes_regenerate(self): | |||
resp = self.client.get(url) | |||
assert old_codes != resp.data['codes'] | |||
|
|||
def test_delete(self): | |||
assert email_log.info.call_count == 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't we want this action to send an email?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ya, I'll add it in a follow-up pr ^
@@ -155,6 +163,8 @@ def test_u2f_remove_device(self): | |||
resp = self.client.delete(url) | |||
assert resp.status_code == 500 | |||
|
|||
assert email_log.info.call_count == 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing a device seems like it should generate an email as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ya, I'll add it in a follow-up pr ^
No description provided.