I agree that catching all instances of IllegalStateException and treating it as
an indicator of client closing the connection is wrong. In fact I think getting
an IllegalStateException when the client closes the connection is wrong
(especially since there's a separate catch block for handling
C
I don't think that this is the correct fix for SYNAPSE-866. My
interpretation is that the original intent of the code you removed was
to avoid logging an exception when the client closes the connection
(which is not a very exceptional situation). The problem is that this
code is triggered also in o