Author: markt Date: Wed Oct 16 11:14:37 2013 New Revision: 1532720 URL: http://svn.apache.org/r1532720 Log: Include socket reference in IOExceptions to make debugging easier
Modified: tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties Modified: tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java?rev=1532720&r1=1532719&r2=1532720&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java (original) +++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java Wed Oct 16 11:14:37 2013 @@ -53,7 +53,7 @@ public class AprServletInputStream exten readLock.lock(); if (wrapper.getBlockingStatus() == block) { if (closed) { - throw new IOException(sm.getString("apr.closed")); + throw new IOException(sm.getString("apr.closed", Long.valueOf(socket))); } result = Socket.recv(socket, b, off, len); readDone = true; @@ -73,7 +73,7 @@ public class AprServletInputStream exten readLock.lock(); writeLock.unlock(); if (closed) { - throw new IOException(sm.getString("apr.closed")); + throw new IOException(sm.getString("apr.closed", Long.valueOf(socket))); } result = Socket.recv(socket, b, off, len); } finally { @@ -102,7 +102,7 @@ public class AprServletInputStream exten throw new EOFException(sm.getString("apr.clientAbort")); } else { throw new IOException(sm.getString("apr.read.error", - Integer.valueOf(-result))); + Integer.valueOf(-result), Long.valueOf(socket))); } } Modified: tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java?rev=1532720&r1=1532719&r2=1532720&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java (original) +++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java Wed Oct 16 11:14:37 2013 @@ -63,7 +63,7 @@ public class AprServletOutputStream exte readLock.lock(); if (wrapper.getBlockingStatus() == block) { if (closed) { - throw new IOException(sm.getString("apr.closed")); + throw new IOException(sm.getString("apr.closed", Long.valueOf(socket))); } return doWriteInternal(b, off, len); } @@ -86,7 +86,7 @@ public class AprServletOutputStream exte readLock.lock(); writeLock.unlock(); if (closed) { - throw new IOException(sm.getString("apr.closed")); + throw new IOException(sm.getString("apr.closed", Long.valueOf(socket))); } return doWriteInternal(b, off, len); } finally { @@ -143,7 +143,7 @@ public class AprServletOutputStream exte throw new EOFException(sm.getString("apr.clientAbort")); } else if (written < 0) { throw new IOException(sm.getString("apr.write.error", - Integer.valueOf(-written))); + Integer.valueOf(-written), Long.valueOf(socket))); } start += written; left -= written; Modified: tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties?rev=1532720&r1=1532719&r2=1532720&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties (original) +++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties Wed Oct 16 11:14:37 2013 @@ -25,7 +25,7 @@ upgrade.sos.writeListener.null=It is ill upgrade.sis.write.ise=It is illegal to call any of the write() methods in non-blocking mode without first checking that there is space available by calling isReady() apr.clientAbort=The client aborted the connection. -apr.read.error=Unexpected error [{0}] reading data from the APR/native socket. -apr.write.error=Unexpected error [{0}] writing data to the APR/native socket. -apr.closed=The socket associated with this connection has been closed. +apr.read.error=Unexpected error [{0}] reading data from the APR/native socket [{1}]. +apr.write.error=Unexpected error [{0}] writing data to the APR/native socket [{1}]. +apr.closed=The socket [{0}] associated with this connection has been closed. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org