Log: strip logged user input for cleaner logs

[kickster97]

WHAT is this pull request doing?

It strip the protocol string from newlines, so we get cleaner logs if users send in bad input.

EX:

Dec 19 06:07:33Z lmq.amqp.connection_factory: "Unexpected protocol '# 003
Dec 19 06:07:33Z *%�
Dec 19 06:07:33Z ', closing socket

HOW can this pull request be tested?

send in a header with a bad protocol, with newlines, and see that the log is not devided.

[kickster97]

This should probably be done in all places where we log user input, but i'll put it up like this so we can discuss.

[carlhoerberg]

only stringify the number of bytes we read, the rest will always be garbage. Then use String#inspect to render \n as "\n", but also all other non non rendable characters, like \t, \0 etc

Suggested change

	log.warn {
	bad_protocol = String.new(proto.to_slice).gsub(/\n/, "\\n")
	"Unexpected protocol '#{bad_protocol}', closing socket"
	}
	log.warn { "Unexpected protocol #{String.new(proto.to_unsafe, count).inspect}, closing socket" }