Fix CRC16 encode : string to bytes

* should use SafeEncoder.encode() instead of String.getBytes()
This commit is contained in:
Jungtaek Lim
2014-09-27 21:46:20 +09:00
parent 7836531ad7
commit a405ce205c
2 changed files with 3 additions and 2 deletions

View File

@@ -59,7 +59,7 @@ public class JedisClusterCRC16 {
}
public static int getCRC16(String key) {
return getCRC16(key.getBytes());
return getCRC16(SafeEncoder.encode(key));
}
}

View File

@@ -9,6 +9,7 @@ import java.util.Map.Entry;
import org.junit.Test;
import redis.clients.util.JedisClusterCRC16;
import redis.clients.util.SafeEncoder;
public class JedisClusterCRC16Test {
@@ -21,7 +22,7 @@ public class JedisClusterCRC16Test {
assertEquals(entry.getValue().intValue(), JedisClusterCRC16.getCRC16(entry.getKey()));
// byte array version
assertEquals(entry.getValue().intValue(), JedisClusterCRC16.getCRC16(entry.getKey().getBytes()));
assertEquals(entry.getValue().intValue(), JedisClusterCRC16.getCRC16(SafeEncoder.encode(entry.getKey())));
}
}