Page MenuHomePhabricator

Add support for using ipv6 address for memcached
Closed, ResolvedPublic

Description

Here https://github.com/miraheze/mediawiki/blob/REL1_37/includes/libs/objectcache/utils/MemcachedClient.php#L788 does not support ipv6.

ip-utils provides a method which could be used:

splitHostAndPort( $ip_and_port )

so this could become

$server = splitHostAndPort( $ip_and_port )
if ($server === false || !isset( $server[1] )) return false;
$ip = $server[0];
$port = $server[1];

I think ?

Event Timeline

Restricted Application added subscribers: Reception123, Aklapper. · View Herald Transcript

Change 751203 had a related patch set uploaded (by Paladox; author: Paladox):

[mediawiki/core@master] MemcachedClient: Add support for ipv6

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

Change 751203 merged by jenkins-bot:

[mediawiki/core@master] MemcachedClient: Add support for ipv6

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

Change 752282 had a related patch set uploaded (by Paladox; author: Paladox):

[mediawiki/core@REL1_37] MemcachedClient: Add support for ipv6

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

Change 752283 had a related patch set uploaded (by Paladox; author: Paladox):

[mediawiki/core@REL1_36] MemcachedClient: Add support for ipv6

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

Change 752284 had a related patch set uploaded (by Paladox; author: Paladox):

[mediawiki/core@REL1_35] MemcachedClient: Add support for ipv6

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

Change 752284 merged by jenkins-bot:

[mediawiki/core@REL1_35] MemcachedClient: Add support for ipv6

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

Change 752282 merged by jenkins-bot:

[mediawiki/core@REL1_37] MemcachedClient: Add support for ipv6

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

Paladox claimed this task.

Change 752283 merged by jenkins-bot:

[mediawiki/core@REL1_36] MemcachedClient: Add support for ipv6

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

Reedy closed this task as Resolved.
Reedy triaged this task as Low priority.
Reedy edited projects, added IPv6; removed Patch-For-Review.