teamhanko / hanko

Authentication and User Management solution for developers. Open source alternative to Clerk & Auth0.
https://hanko.io
Other
7.29k stars 899 forks source link

Email security notifications #1030

Open FlxMgdnz opened 1 year ago

FlxMgdnz commented 1 year ago

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

irby commented 1 year ago

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

irby commented 1 year ago

@FlxMgdnz I’ll take on this item

FlxMgdnz commented 1 year ago

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

prashantug21 commented 1 year ago

@FlxMgdnz Is this issue available?

Fer-Win commented 1 year ago

@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 commented 1 year ago

@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 :)

Fer-Win commented 1 year ago

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

irby commented 1 year ago

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

FlxMgdnz commented 1 year ago

@irby yes we should log the emails.

SMTP settings refactoring makes sense imho.

irby commented 1 year ago

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

FlxMgdnz commented 1 year ago

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

wrobat commented 1 month ago

/assign

irby commented 1 month ago

I'm willing to take myself off this ticket. I haven't actioned on this in some time. @FlxMgdnz @wrobat

Aashana101 commented 3 weeks ago

@FlxMgdnz kindly assign this to me