zero length 'chunk' should not be an error

[laprej]

I don't believe that a zero length chunk should trigger an exception. Rather it's the proper way to indicate the other end of the TCP connection has shutdown. I'm not sure in the end it really changes much: kazoo may treat this as an exceptional event which is a fair interpretation. But the exception name / text is misleading as well. ConnectionClosed('socket closed') might be more appropriate.

kazoo/kazoo/protocol/connection.py

Lines 259 to 260 in 6337fd6

	if chunk == b'':
	raise ConnectionDropped('socket connection broken')

[StephenSorriaux]

Hello,

Good point. I will check if the change is that straightforward as it seems.