search

ryandotsmith / log-shuttle

New Repository: https://github.com/heroku/log-shuttle
https://github.com/heroku/log-shuttle
22 stars 6 forks source link

Fix issue where we added empty packets to the output buffer. #25

Closed ryandotsmith closed 11 years ago

ryandotsmith commented 11 years ago

0 is not a valid syslog packet length. Thus if we detect this case, we should not add the empty packet to the output buffer.

Related: #23

@archaelus look good to you?

archaelus commented 11 years ago

I was going to say 'maybe you should log blanks' but that makes no sense on reflection.

This patch looks good to me: +1.