fluencelabs/redis
1
0
Fork 0
mirror of https://github.com/fluencelabs/redis synced 2025-03-19 09:00:51 +00:00
Code Issues Projects Releases Wiki Activity

Sentinel: use comma as separator to publish hello messages.

Browse Source 
We use comma to play well with IPv6 addresses, but the implementation is
still able to parse the old messages separated by colons.
This commit is contained in:
antirez 2013-07-11 16:31:39 +02:00
parent 4735f03c5a
commit 3fc7f324d2
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/sentinel.c
View File

@ -1741,9 +1741,13 @@ void sentinelReceiveHelloMessages(redisAsyncContext *c, void *reply, void *privd
{ {
int numtokens, port, removed, canfailover; int numtokens, port, removed, canfailover;
/* Separator changed from ":" to "," in recent versions in order to
* play well with IPv6 addresses. For now we make sure to parse both
* correctly detecting if there is "," inside the string. */
char *sep = strchr(r->element[2]->str,',') ? "," : ":";
char **token = sdssplitlen(r->element[2]->str, char **token = sdssplitlen(r->element[2]->str,
r->element[2]->len, r->element[2]->len,
":",1,&numtokens); sep,1,&numtokens);
sentinelRedisInstance *sentinel; sentinelRedisInstance *sentinel;
if (numtokens == 4) { if (numtokens == 4) {
@ -1841,8 +1845,7 @@ void sentinelPingInstance(sentinelRedisInstance *ri) {
if (anetSockName(ri->cc->c.fd,ip,sizeof(ip),NULL) != -1) { if (anetSockName(ri->cc->c.fd,ip,sizeof(ip),NULL) != -1) {
char myaddr[REDIS_IP_STR_LEN+128]; char myaddr[REDIS_IP_STR_LEN+128];
// FIXME: IPv6 will break this due to nested : characters -geoffgarside snprintf(myaddr,sizeof(myaddr),"%s,%d,%s,%d",
snprintf(myaddr,sizeof(myaddr),"%s:%d:%s:%d",
ip, server.port, server.runid, ip, server.port, server.runid,
(ri->flags & SRI_CAN_FAILOVER) != 0); (ri->flags & SRI_CAN_FAILOVER) != 0);
retval = redisAsyncCommand(ri->cc, retval = redisAsyncCommand(ri->cc,