diff options
| author | Dana Powers <dana.powers@rd.io> | 2015-11-28 19:34:37 +0800 |
|---|---|---|
| committer | Zack Dever <zack.dever@rd.io> | 2015-12-04 11:25:39 -0800 |
| commit | f08775a6198cd16a7bc9ec93ffd057f65064ec54 (patch) | |
| tree | e46e8df3d16e7baaf82a5d3dc260499955a7d39b /kafka/protocol/legacy.py | |
| parent | dc94b5fe9f3f93bf6f2235d7f65c62fcf0a2a996 (diff) | |
| download | kafka-python-f08775a6198cd16a7bc9ec93ffd057f65064ec54.tar.gz | |
Switch crc32 back to signed integer -- this is consistent with protocol encoding spec
Diffstat (limited to 'kafka/protocol/legacy.py')
| -rw-r--r-- | kafka/protocol/legacy.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kafka/protocol/legacy.py b/kafka/protocol/legacy.py index d7ac50a..db9f3e0 100644 --- a/kafka/protocol/legacy.py +++ b/kafka/protocol/legacy.py @@ -109,7 +109,7 @@ class KafkaProtocol(object): write_int_string(message.value) ]) crc = crc32(msg) - msg = struct.pack('>I%ds' % len(msg), crc, msg) + msg = struct.pack('>i%ds' % len(msg), crc, msg) else: raise ProtocolError("Unexpected magic number: %d" % message.magic) return msg @@ -159,7 +159,7 @@ class KafkaProtocol(object): The offset is actually read from decode_message_set_iter (it is part of the MessageSet payload). """ - ((crc, magic, att), cur) = relative_unpack('>IBB', data, 0) + ((crc, magic, att), cur) = relative_unpack('>iBB', data, 0) if crc != crc32(data[4:]): raise ChecksumError("Message checksum failed") |