-
Notifications
You must be signed in to change notification settings - Fork 797
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
Email security notifications #1030
Comments
@FlxMgdnz Would we want to configure each of these scenarios individually? |
@FlxMgdnz I’ll take on this item |
@FlxMgdnz Is this issue available? |
@FlxMgdnz If no is currently working on it or you would want someone else to work on it. Let me try it please..Assign it to me... |
@irby ok fine, let me know if any other issue opens up |
@FlxMgdnz I'm assuming emails sent out should be recorded in an audit log, correct? Do we want to refactor the SMTP settings? Currently, these configuration settings are setup under passcode, but obviously as we include security notifications these will no longer only apply to passcodes. |
@irby yes we should log the emails. SMTP settings refactoring makes sense imho. |
@FlxMgdnz It'll make sense to break out the SMTP settings refactor into its own PR. Do you want me to create an issue for this change, or can I link it to this issue? |
Right, that makes sense. Linking to this issue/comment should be sufficient. Thanks @irby ! |
Currently, Hanko backend only sends emails for passcode authentication.
Add a new config option to enable / disable account security notifications.
If enabled, send security notification emails to the user's primary email address on
The text was updated successfully, but these errors were encountered: