I was going to comment that isDebugEnabled() is not needed when you are not constructing the argument to debug() - that test is generally to avoid the overhead of building an argument and then have the debug() method toss it away. When you pass in a simple string, it's redundant, because that's the first test in debug().

That said, I should enhance this message to include the connectionId, in which case the isDebugEnabled() call __is__ necessary.