Page MenuHomePhorge

Log failure reason
ClosedPublic

Authored by mollekopf on Sep 2 2024, 2:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 1, 9:39 PM
Unknown Object (File)
Fri, Nov 1, 3:41 AM
Unknown Object (File)
Thu, Oct 31, 4:58 PM
Unknown Object (File)
Mon, Oct 28, 1:30 PM
Unknown Object (File)
Sun, Oct 27, 2:45 PM
Unknown Object (File)
Sun, Oct 27, 2:45 PM
Unknown Object (File)
Sun, Oct 27, 2:45 PM
Unknown Object (File)
Thu, Oct 24, 8:27 AM
Subscribers

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rK0bb0414fd53e: Log failure reason

Diff Detail

Repository
rK kolab
Branch
master
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 48903
Build 18360: arc lint + arc unit

Event Timeline

mollekopf created this revision.

So I don't have to go modify production code to figure out that geolocation somehow blocks the login attempt.

mollekopf added a reviewer: Restricted Project.Sep 2 2024, 2:57 PM
machniak added inline comments.
src/app/Http/Controllers/API/AuthController.php
86

This message should have some context. Maybe add "Auth: " prefix to it or something.

Also, I'm not sure it makes sense to log anything at this point. Do we want to log user input on every login?

src/app/Http/Controllers/API/AuthController.php
86

Maybe it doesn't need to be a warning. When troubleshooting it's very useful to get an indication what is going wrong, and a generic 401 is rather useless, so in general I think yes, we should be logging something. It may be too noisy by default though.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 6 2024, 2:18 PM
Closed by commit rK0bb0414fd53e: Log failure reason (authored by mollekopf). · Explain Why
This revision was automatically updated to reflect the committed changes.