Page MenuHomePhabricator

Add variables in Abusefilter to detect/block thanks
Open, Stalled, LowPublicFeature

Description

Hi. To fight harassment, is it possible to add variables to Special:Abusefilter regarding "thanks"? One could be "thanks sender username", another "thanks recipient username". The goal would be to avoid abuses of thanks functionality by some banned users.

Example: in a fr-wiki case, it would prevent users with a name ending by -ah from thanking a sysop in order to harass him. (The sysop had to disable the thanks functionality.)

Regards.

Details

Related Gerrit Patches:

Event Timeline

Restricted Application added a project: Growth-Team. · View Herald TranscriptOct 18 2019, 1:25 PM
Restricted Application added subscribers: MGChecker, Aklapper. · View Herald Transcript
Jules78120 updated the task description. (Show Details)Oct 18 2019, 1:28 PM

Somewhat related to T177780. I've just found an incomplete comment on that task where I was saying that filtering thank actions in AF would be technically difficult, but didn't say why :(

Masumrezarock100 added a comment.EditedDec 26 2019, 7:21 PM

Please implement this. One WMF banned editor has a habit of abusing thanks using his socks and I am tired of being harassed by him.

Xaosflux changed the subtype of this task from "Task" to "Feature Request".Dec 26 2019, 7:22 PM
Restricted Application added a project: User-DannyS712. · View Herald TranscriptDec 26 2019, 7:29 PM
DannyS712 triaged this task as Low priority.Dec 26 2019, 7:37 PM

Thanks creates an echo notification using EchoEvent::create, which runs the BeforeEchoEventInsert hook before inserting the EchoEvent. AbuseFilter should process this hook:

  • If type = edit-thank
  • Extract User sending the thanks from agent
  • Extract target user id from extra['thanked-user-id']
  • Since thanks is generally private (in terms of what edit/log entry the thanks is for), don't process the title or other event properties

Change 560807 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[integration/config@master] Add Echo extension as a dependency for AbuseFilter

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

In progress (it'll take a while), but noticed that, even if the thanks are prevented, it'll still act like they went through (just that the recipient doesn't get the eventual notification). Filed T241474

This requires designing how to present warnings / disallow messages, or setting it to not show a message, not allow warnings, and fail silently on disallow

This requires designing how to present warnings / disallow messages, or setting it to not show a message, not allow warnings, and fail silently on disallow

This may be one of the reasons which led me to write T235873#5586905.

This requires designing how to present warnings / disallow messages, or setting it to not show a message, not allow warnings, and fail silently on disallow

This may be one of the reasons which led me to write T235873#5586905.

Filed as T241550

DannyS712 changed the task status from Open to Stalled.Tue, Dec 31, 11:24 PM

Stalling on subtasks - until there is a way to show a message, and until the thanks api recognizes that it shouldn't assume the thanks we successfully delivered, progress can't really be made