Log messages for successful login #136

Closed
opened 2026-02-04 17:50:29 +03:00 by OVERLORD · 5 comments
Owner

Originally created by @RomanHargrave on GitHub (Nov 20, 2018).

We have a login message for failed login attempts, but no counterpart for successful login attempts.

It should include username, and IP address/host, as with the failed attempts.

Originally created by @RomanHargrave on GitHub (Nov 20, 2018). We have a login message for failed login attempts, but no counterpart for successful login attempts. It should include username, and IP address/host, as with the failed attempts.
OVERLORD added the enhancementgood first issue labels 2026-02-04 17:50:29 +03:00
Author
Owner

@mprasil commented on GitHub (Nov 20, 2018):

This should be fairly easy to add.

@mprasil commented on GitHub (Nov 20, 2018): This should be fairly easy to add.
Author
Owner

@RomanHargrave commented on GitHub (Nov 20, 2018):

I'll probably get it when I get home.

@RomanHargrave commented on GitHub (Nov 20, 2018): I'll probably get it when I get home.
Author
Owner

@Algebro7 commented on GitHub (Dec 11, 2018):

I'm guessing the rocket-0.4 branch would be the best place for this due to rocket logging features added in 2fde4e6?

@Algebro7 commented on GitHub (Dec 11, 2018): I'm guessing the rocket-0.4 branch would be the best place for this due to rocket logging features added in 2fde4e6?
Author
Owner

@dani-garcia commented on GitHub (Dec 11, 2018):

Yes, that branch is probably going to be merged sometime this week, so it's probably the best place to make changes to.

@dani-garcia commented on GitHub (Dec 11, 2018): Yes, that branch is probably going to be merged sometime this week, so it's probably the best place to make changes to.
Author
Owner

@mprasil commented on GitHub (Dec 12, 2018):

This is in latest beta, I think we can now close this.

@mprasil commented on GitHub (Dec 12, 2018): This is in latest beta, I think we can now close this.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/vaultwarden#136