Last modified: 2014-01-08 21:43:56 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T52894, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 50894 - Throttling doesn't work without object caching
Throttling doesn't work without object caching
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
AbuseFilter (Other open bugs)
master
All All
: High major with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 55816 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-07 19:18 UTC by Kazimierz
Modified: 2014-01-08 21:43 UTC (History)
8 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Kazimierz 2013-07-07 19:18:34 UTC
A simple filter: 
length(added_links) > 0
works fine.

But if I try to add throttling, it stops working at all. I tried with the options:
Number of actions to allow: 2
Period of time: 3600 seconds
Group throttle by: user
Action: Block the user and/or IP from editing.

Then I tried adding links several times to a single one and different articles by the same (registered) user, but the user has not been blocked. Also tried different actions, like Tag the edit for further review, and Trigger these actions after giving the user a warning. Nothing works, the warning is not been shown too. And the hit count on the filter stays at 0.
Comment 1 Andre Klapper 2013-07-07 21:09:57 UTC
Thanks for taking the time to report this!

Which versions of MediaWiki and which version of AbuseFilter is this about?
Also see http://www.mediawiki.org/wiki/Manual:How_to_debug
Comment 2 Kazimierz 2013-07-08 04:23:08 UTC
> Which versions of MediaWiki and which version of AbuseFilter is this about?

Sorry, forgot to add this: MW 1.21, AbuseFilter master.

I will try to debug when I have some time.
Comment 3 Kazimierz 2013-07-09 19:15:13 UTC
I found out that the internal count of filter hits read from Memcached is always 0. I don't have Memcached enabled.

isThrottled function (AbuseFilter.class.php)
$key = 'wiki:abusefilter:throttle:2:user:3' (seems legit)
$count = 0 (always). 

I think the problem is that AbuseFilter always uses Memcached for throttle, even if it's not enabled.
Comment 4 Gerrit Notification Bot 2013-10-28 05:49:47 UTC
Change 92266 had a related patch set uploaded by Kaldari:
Only allow throttling if object caching is enabled...

https://gerrit.wikimedia.org/r/92266
Comment 5 Jon 2013-10-31 21:43:21 UTC
*** Bug 55816 has been marked as a duplicate of this bug. ***
Comment 6 Gerrit Notification Bot 2013-11-01 12:26:36 UTC
Change 92266 merged by jenkins-bot:
Only allow throttling if object caching is enabled...

https://gerrit.wikimedia.org/r/92266
Comment 7 Marcin Cieślak 2013-11-01 12:47:36 UTC
Probably "throttle" should be taken out of the user interface if there is no memcached enabled?

Also "blockautopromote" is using memcached only and I believe this is actually bug 20487
Comment 8 Jackmcbarn 2014-01-08 21:37:37 UTC
The "fix" for this appears to be a band-aid. The throttle should be made to work without object caching, rather than simply being hidden when it is not present.
Comment 9 Aaron Schulz 2014-01-08 21:43:56 UTC
(In reply to comment #8)
> The "fix" for this appears to be a band-aid. The throttle should be made to
> work without object caching, rather than simply being hidden when it is not
> present.

Probably some code using $wgMemc can just use wfGetCache( CACHE_ANYTHING );

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links