Page MenuHomePhabricator

Investigation: Why doesn't LoginNotify work well with MediaWiki Vagrant + the Echo role
Closed, ResolvedPublic5 Estimated Story Points

Description

Both Leon and Niharika have reported problems with using LoginNotify with MediaWiki Vagrant + Echo role. Figure out why it's causing errors and how to address those (and make sure they aren't problems that will occur in production). This may be due to the fact that LoginNotify interfaces with CentralAuth in mutli-wiki environments (and the code that interfaces with CentralAuth may be out of date).

Event Timeline

kaldari renamed this task from Investigation: Why doesn't LoginNotify work well with MediaWiki Vagrant + the Echo roll to Investigation: Why doesn't LoginNotify work well with MediaWiki Vagrant + the Echo role.Mar 21 2017, 11:16 PM
kaldari updated the task description. (Show Details)
kaldari set the point value for this task to 5.
kaldari moved this task from New & TBD Tickets to Needs Discussion on the Community-Tech board.
kaldari moved this task from Needs Discussion to Up Next (June 3-21) on the Community-Tech board.

I figured out that the missing piece for getting it to work both locally and on commtech wiki was a composer update. The strange bit is I did run vagrant git-update for both which I am pretty certain does a composer update. Anyway, doing it manually fixed the problem. Now it does work for me. We should create a vagrant role for LoginNotify to avoid this problem for anyone who wants to contribute to the extension.

Also, I found a bug. If I do failed attempts to login to an account twice, it issues me two notifications:

Screen Shot 2017-04-03 at 4.50.07 PM.png (305×499 px, 51 KB)

Tried again with three failed attempts:

Screen Shot 2017-04-03 at 4.51.53 PM.png (398×505 px, 73 KB)

I've filed tickets for both: T162103: Create a vagrant role for installing LoginNotify with vagrant and T162104: Fix multiple/repetitive notifications with LoginNotify