Page MenuHomePhabricator

Set-up EventLogging to see how often blocked users are blocked via cookie
Closed, ResolvedPublic3 Estimated Story Points

Description

Set up EventLogging to see how often blocked users are blocked from performing an action via cookie.

This will help us to make sure that this technique works, and actually keeps blocked people out. It'll also be a signal that tells us if we're blocking too many people. The expected range is somewhere between "some" and "holy cow, that's crazy". This is an imperfect metric, but we live in an imperfect world.

See parent task (T5233) for details.

Event Timeline

kaldari set the point value for this task to 3.
DannyH raised the priority of this task from Low to Medium.Sep 20 2016, 11:52 PM
DannyH removed a project: Community-Tech-fixes.
DannyH updated the task description. (Show Details)
DannyH moved this task from New & TBD Tickets to Up Next on the Community-Tech board.

Apologies for being technically illiterate, but could the impact of cookie blocks be seen through the front-end interface? This would be useful for those actually giving the blocks, or for CheckUsers investigating block evasion.

Niharika subscribed.

I'll like to take this on, unless someone else wanted to. It's a good opportunity to learn how EventLogging works. :)

Well, it's blocked until T5233 is done and merged. Doesn't need to wait for it to be deployed.

Change 320239 had a related patch set uploaded (by Niharika29):
Add eventlogging for checking how often users are blocked from editing due to presence of BlockID cookie

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

Change 320405 had a related patch set uploaded (by Niharika29):
Add var to User class to store block type for eventlogging purposes

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

Change 320405 merged by jenkins-bot:
Add var to User class to store block type for eventlogging purposes

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

Core change is merged. Just the hook handler left to go.

Change 320239 merged by jenkins-bot:
Add eventlogging for checking how often users are blocked from editing due to presence of BlockID cookie

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

The logging does not handle any other blockTrigger value than the ones in the enum declaration. That will be problematic with https://gerrit.wikimedia.org/r/#/c/324214. Either that field should be defined as a string, or as a boolean and only log whether the trigger was a cookie or not.

Change 330890 had a related patch set uploaded (by Niharika29):
Replace block trigger string by bool to indicate whether it was a cookie block

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

I'll amend the schema and events hook if the above^ looks right. @Tgr @kaldari

Moving back to sprint so I don't forget to do this.

Change 330890 abandoned by Niharika29:
Replace block trigger string by bool to indicate whether it was a cookie block

Reason:
Per Gergo's comment.

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

Change 332769 had a related patch set uploaded (by BryanDavis):
Only record cookie-block events in eventlogging.

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

Change 332769 merged by jenkins-bot:
Only record cookie-block events in eventlogging.

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