Last modified: 2007-07-05 19:49:30 UTC
Created attachment 3859 [details] patch with 'patch -P0 < ConfirmEditWhitelistIP.patch' from /extensions/ConfirmEdit This is a feature I found useful on a Wiki I run. In my case, to allow all edits from my school's subnet. It works by specifying an array of regexes to match against IP addresses, if the address matches the edit is allowed.
Seems reasonable.
It seems a bit odd to me to use a regex for an IP list. It would seem cleaner to use existing (hopefully ;) functions for CIDR IP ranges.
Created attachment 3868 [details] Alternate patch to use CIDR ranges instead of regexs There's nothing in PHP, but there seems to be something buried in /includes/IP.php ;)
Looks good -- works for individual IPs as well as ranges, nice! Applied in r23758, with some tweaks to the doc comment.