The onAuthManagerLoginAuthenticateAudit() hook call back in CheckUser should be edited such that if the login attempt was successful, the row entered in cu_changes would indicate the user name and ID. In cases where the login was unsuccessful, current code (which only attributes the event to the IP address) is appropriate.
Originally reported on the CheckUser listserv.
Technical debt was introduced in a14842e7
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Successful login attemps should be logged for the user account | mediawiki/extensions/CheckUser | master | +16 -11 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Huji | T253802 Configure WMF wikis to log login attempts in CheckUser | |||
| Resolved | Huji | T253946 Successful login attemps should be logged for the user account, not just the IP |
Change 599486 had a related patch set uploaded (by Huji; owner: Huji):
[mediawiki/extensions/CheckUser@master] Successful login attemps should be logged for the user account
Change 599492 had a related patch set uploaded (by Huji; owner: Huji):
[operations/mediawiki-config@master] Set wgCheckUserLogLogins to true by default
Change 599486 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Successful login attemps should be logged for the user account