Comma-delimited servers list not supported in memcached caches `LOCATION`

[Ed Morley]

Currently Django's memcached backends only support a semicolon delimited servers list in the cache config, such as:

CACHES = {
    'default': {
        'LOCATION': 'server1.foo.com;server2.foo.com',
    },
}

This is because the backend ​just does:

    self._servers = server.split(';')

However cloud memcached offerings such as Memcachier and Memcached Cloud typically use a comma-delimited list in the automatically set environment variable, eg:

MEMCACHIER_SERVERS='foo.us-east-5.heroku.prod.memcachier.com:11211,bar.us-east-5.heroku.prod.memcachier.com:11211'

Given that the caches LOCATION parameter contains only domain name and port (and not say passwords), it should never contain a comma - so we should be fine to also split on them too.

[Tim Graham]

I guess server.replace(',', ';').split(';') should work.

[Ed Morley]

Ah I have this finished locally already - sorry thought I'd set the ticket assignee already!

[Olexander Yermakov]

Ok. Not a problem.

[Ed Morley]

​PR

[Tim Graham]

Pending a few cosmetic edits.

[Tim Graham <timograham@…>]

In d8ef5b0e:

Fixed #27152 -- Supported comma delimiter in memcached LOCATION string.