Re: svn commit: r1504950 - /synapse/trunk/java/modules/transports/core/nhttp/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreNIOSender.java

2013-08-03 Thread Hiranya Jayathilaka
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

Re: svn commit: r1504950 - /synapse/trunk/java/modules/transports/core/nhttp/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreNIOSender.java

2013-08-03 Thread Andreas Veithen
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