Page MenuHomePhabricator
Create Task
Maniphest T207540

"Login for $1 succeeded" should include IP information
Closed, ResolvedPublic
Actions

Description

2018-10-20 12:06:19 [W8saOQpAICoAABOw3bkAAADF] mw1321 enwikiquote 1.32.0-wmf.26 authentication INFO: Login for Miszatomic succeeded {"user":"Miszatomic"}

This should include IP information

Related Objects
Search...

StatusAssignedTask
Restricted Task
 ResolvedLegoktmT207540 "Login for $1 succeeded" should include IP information

Event Timeline

Reedy triaged this task as High priority.Oct 20 2018, 12:13 PM
Reedy created this task.
Reedy created this object with visibility "Custom Policy".
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 20 2018, 12:13 PM
Legoktm added a subscriber: Legoktm.Oct 20 2018, 12:35 PM

0001-SECURITY-Include-IP-address-in-Login-for-1-succeeded.patch1 KBDownload

Untested.

Reedy added a comment.Oct 20 2018, 12:42 PM
In T207540#4682491, @Legoktm wrote:

0001-SECURITY-Include-IP-address-in-Login-for-1-succeeded.patch1 KBDownload

Untested.

Guess it should be clientip to be consistent :)

Reedy added a comment.Oct 20 2018, 12:44 PM

Any objections for just putting this into gerrit?

Reedy closed this task as Resolved.Oct 20 2018, 5:20 PM
Reedy assigned this task to Legoktm.
Reedy changed the visibility from "Custom Policy" to "Public (No Login Required)".
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL