hanko icon indicating copy to clipboard operation
hanko copied to clipboard

Email security notifications

Open FlxMgdnz opened this issue 1 year ago • 13 comments

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

  • Password changed
  • Email address added
  • Primary email address changed (use primary email before the change)
  • Passkey created

FlxMgdnz avatar Sep 21 '23 07:09 FlxMgdnz

@FlxMgdnz Would we want to configure each of these scenarios individually?

irby avatar Sep 23 '23 01:09 irby

@FlxMgdnz I’ll take on this item

irby avatar Sep 26 '23 23:09 irby

@FlxMgdnz Would we want to configure each of these scenarios individually?

@irby I think it would be better to be able to toggle each warning individually, yes.

FlxMgdnz avatar Sep 27 '23 08:09 FlxMgdnz

@FlxMgdnz Is this issue available?

prashantug21 avatar Oct 01 '23 06:10 prashantug21

@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...

Fer-Win avatar Oct 10 '23 03:10 Fer-Win

@Fer-Win I am currently working on this one. About 80% done, ran into an issue I'm working to resolve before I can get a PR up :)

irby avatar Oct 10 '23 13:10 irby

@irby ok fine, let me know if any other issue opens up

Fer-Win avatar Oct 10 '23 13:10 Fer-Win

@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 avatar Oct 21 '23 17:10 irby

@irby yes we should log the emails.

SMTP settings refactoring makes sense imho.

FlxMgdnz avatar Oct 21 '23 18:10 FlxMgdnz

@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?

irby avatar Oct 21 '23 19:10 irby

Right, that makes sense. Linking to this issue/comment should be sufficient. Thanks @irby !

FlxMgdnz avatar Oct 21 '23 20:10 FlxMgdnz