Page MenuHomePhabricator

Test LoginNotify Extension on Beta Cluster
Closed, ResolvedPublic3 Estimated Story Points

Description

Turn on the LoginNotify Extension on the Beta Cluster and test its various functions. See https://www.mediawiki.org/wiki/Extension:LoginNotify.

Also, see if you can answer the question at T158871#3053086.

Event Timeline

kaldari set the point value for this task to 3.Feb 23 2017, 5:55 PM
kaldari raised the priority of this task from Medium to High.Mar 9 2017, 6:16 PM

Setting to high priority since launching LoginNotify is one of our quarterly goals.

Change 345726 had a related patch set uploaded (by Niharika29):
[operations/mediawiki-config@master] Test LoginNotify on Beta cluster

https://gerrit.wikimedia.org/r/345726

Scheduled for swat this evening.

Change 345726 merged by jenkins-bot:
[operations/mediawiki-config@master] Test LoginNotify on Beta cluster

https://gerrit.wikimedia.org/r/345726

Mentioned in SAL (#wikimedia-operations) [2017-04-03T23:10:30Z] <thcipriani@tin> Synchronized wmf-config: SWAT: [[gerrit:345726|Test LoginNotify on Beta cluster]] T158878 (duration: 00m 46s)

Change 346228 had a related patch set uploaded (by Reedy):
[operations/mediawiki-config@master] Disable LoginNotify on wikis that don't have Echo

https://gerrit.wikimedia.org/r/346228

Change 346228 merged by jenkins-bot:
[operations/mediawiki-config@master] Disable LoginNotify on wikis that don't have Echo

https://gerrit.wikimedia.org/r/346228

Mentioned in SAL (#wikimedia-operations) [2017-04-04T01:31:30Z] <reedy@tin> Synchronized wmf-config/InitialiseSettings-labs.php: Disable LoginNotify on wikis that have no Echo T158878 (duration: 00m 44s)

As the commit says... You can't enable the extension places that don't have Echo enabled.... :)

It's not possible to have "global" notifications in Echo, i.e. your notification has to be tied to a wiki. Here's the current implementation:

What it looks like on the wiki the logins were attempted on:

Screen Shot 2017-04-05 at 11.22.19 AM.png (530×1 px, 91 KB)

What it looks like on any other wiki on the cluster:

Screen Shot 2017-04-05 at 11.22.07 AM.png (706×1 px, 107 KB)

Note that clicking on the "There have been 4 fails attempts to log in to you ..." message does not do anything. We should make it link to the home wiki because otherwise the message is not completely visible.

Latest screenshots.

On the wiki where the login attempt happened:

Screen Shot 2017-04-17 at 9.58.34 AM.png (214×509 px, 35 KB)

On a different wiki:

Screen Shot 2017-04-17 at 10.00.44 AM.png (304×515 px, 34 KB)

Screen Shot 2017-04-17 at 10.05.31 AM.png (308×612 px, 43 KB)

There are two things that are kind of bothering me:

  1. Clicking on "There have been 4 login attempts..." message on another wiki (2nd image) doesn't do anything but it does show you the click cursor. This is sort of problematic. To actually read the message, the user needs to click "English Wikipedia" (see 2nd image) which is not so intuitive.
  2. Keeping the message exactly same for both new IPs and known IPs doesn't make a lot of sense. If the person accidentally made a password error logging into their account and the night before someone had attempted to login to their account, they'd likely dismiss it for being their own error. How about we add "from a new computer/IP" and "from a known computer/IP" to the new/known IP messages? Or something along those lines. If it was me, I'd even like to see the IP from where the attempt happened but that doesn't make sense for most people. What Google/FB/Twitter do is to show an approximate location from where the edit might have happened. We can think about something like that in the future.

Screen Shot 2017-04-17 at 10.18.32 AM.png (154×555 px, 37 KB)

Screen Shot 2017-04-17 at 10.16.44 AM.png (367×649 px, 40 KB)

Screen Shot 2017-04-17 at 10.18.04 AM.png (349×420 px, 149 KB)

@Niharika: The first one sounds like a problem inherent in global notifications. Let's talk to the collab team and see what they say. For the second one, what do you think about just adding "from a new device" to the unknown notification (similar to LinkedIn's wording)?

@Niharika: The first one sounds like a problem inherent in global notifications. Let's talk to the collab team and see what they say. For the second one, what do you think about just adding "from a new device" to the unknown notification (similar to LinkedIn's wording)?

Sounds good. I forgot to mention, for the first problem, if you hover on the text long enough, you will see the full string. And we do have the option of creating a link from it but to where should we link it is the question.

Change 349264 had a related patch set uploaded (by Niharika29):
[mediawiki/extensions/LoginNotify@master] Change wording to distinguish between new and known devices

https://gerrit.wikimedia.org/r/349264

Change 349264 merged by jenkins-bot:
[mediawiki/extensions/LoginNotify@master] Change wording to distinguish between new and known devices. Add PLURAL for single/multiple attempts. Separate notifications for new/known devices.

https://gerrit.wikimedia.org/r/349264