Page MenuHomePhabricator

Conflict between the Abuse Filter and the Captcha
Closed, ResolvedPublic

Description

When an anonymous user attempted to add a external link to an article, he is required to solve the captcha, and when submitting his edit again, he triggered the local Abuse Filter and got a warning. But this is a false-positive. And after clicking the "submit" button for a second time, he is required to solve the captcha again. Seems this is a problem between the AF and the captcha.

Perhaps the user should only be required to solve the captcha once, not twice.

Best regards.

See Also:

Event Timeline

bzimport raised the priority of this task from to High.Nov 21 2014, 10:51 PM
bzimport added a project: AbuseFilter.
bzimport set Reference to bz20661.
bzimport added a subscriber: Unknown Object (MLST).

matthew.britton wrote:

The way we treat anonymous users nowadays, sometimes I'm amazed we even still get contributions...

(In reply to comment #0)

When an anonymous user attempted to add a external link to an article, .. he
triggered the local Abuse Filter and got a warning.

Which project was this on? Which edit filter was triggered?

(In reply to comment #2)

(In reply to comment #0)

When an anonymous user attempted to add a external link to an article, .. he
triggered the local Abuse Filter and got a warning.

Which project was this on? Which edit filter was triggered?

It happens on pt.wikipedia, for instance, where CAPTCHA is required at every edit until you are autoconfirmed.

I confirm this happened a lot on Portuguese Wikipedia.

swalling wrote:

Is this still happening? Can we just turn off the AbuseFilter for this?

swalling wrote:

If this is still happening, it's completely unacceptable. Setting to a higher priority.

This seems similar to bug 10729: The fact that the user already solved the CAPTCHA is forgotten when another hook (AbuseFilter in this case, edit summary reminder in the other) prevents you from saving, which means you have to solve the CAPTCHA twice.

This is still happening (see this false positive report: https://en.wikipedia.org/wiki/Wikipedia:Edit_filter/False_positives/Reports#James_Roten and this VPT thread: https://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Autobiography_filter)

The issue here, which I believe is the same, is that a user failed the captcha a few times but AF treated each failure as an edit attempt, counting them towards the throttle limit.

This comment was removed by Florian.

Change 295605 had a related patch set uploaded (by Florianschmidtwelzow):
Don't abort EditFilterMergedContent hook

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

Change 295605 abandoned by Florianschmidtwelzow:
Don't abort EditFilterMergedContent hook

Reason:
Alread fixed with I27f244631e9dcd160bffff70349e5034f2a537ea

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

Feel free to reopen, if the problem still exists :)

I guess it does: T151116

Actually, AF interaction with Captcha and Summary reminder is quite awful. In the next days I'll try to put together all these conflicts in a parent task, then we should think of a way to solve this, which may require some core changes.