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

Reply via email to