Page MenuHomePhabricator

CheckUser should take over Block::doRetroactiveAutoblock() lookups
Closed, ResolvedPublic

Description

In internal discussions re: IPv6 deployment for Wikimedia, we found that some of our large sites such as en.wikipedia.org have a recentchanges.rc_ip column that is not wide enough to hold IPv6 addresses.

Since CheckUser has long since taken over all other roles rc_ip used to hold for logging, if we can let CheckUser also handle this we'll be able to simply turn off $wgPutIPInRC and leave rc_ip entries empty without having to worry about it.

Will need backport to 1.17.


Version: unspecified
Severity: enhancement

Details

Reference
bz29330

Event Timeline

bzimport raised the priority of this task from to High.Nov 21 2014, 11:29 PM
bzimport added a project: CheckUser.
bzimport set Reference to bz29330.

Is there already a way to add block options via hook? Maybe we could just defined an autoblock handler callback (setable via extension) and only show the option if it's set.

happy.melon.wiki wrote:

The functionality isn't currently in either the old or new blocking code, but it could be much more easily added in the new (1.18+) code.

Added to code in 92220. Must be configured to use.