Page MenuHomePhabricator

IP Zone issue
Closed, ResolvedPublic

Description

Author: nkuznia

Description:
I encountered a similar error in v1.18.2 While support for IPv6 was added, support for the IP zone was not, for my application, I simply trimmed the zone information off as follows: line 659+ in /includes/IP.php:

public static function canonicalize( $addr ) {

   $addr = preg_replace('/\%.*/i',,$addr); #line I added to remove zone info

if ( self::isValid( $addr ) ) { return $addr; }

Version: 1.18.x
Severity: major

Details

Reference
bz35738

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:12 AM
bzimport set Reference to bz35738.
bzimport added a subscriber: Unknown Object (MLST).

nkuznia wrote:

$addr = preg_replace('/\%.*/i','',$addr); #line I added to remove zone info

prev dropped some quotes on copy

sumanah wrote:

Nick, would you be interested in suggesting your change via our source control system?

https://www.mediawiki.org/wiki/How_to_become_a_MediaWiki_hacker

Submitted to gerrit as Icccf1097

sumanah wrote:

Patch merged. Thanks!