Author: markt
Date: Wed Oct 16 11:23:20 2013
New Revision: 1532726

URL: http://svn.apache.org/r1532726
Log:
Include socket reference in IOExceptions to make debugging easier

Modified:
    tomcat/tc7.0.x/trunk/   (props changed)
    
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java
    
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java
    
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties

Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
  Merged /tomcat/trunk:r1532720

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java?rev=1532726&r1=1532725&r2=1532726&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletInputStream.java
 Wed Oct 16 11:23:20 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 {
@@ -100,7 +100,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/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java?rev=1532726&r1=1532725&r2=1532726&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AprServletOutputStream.java
 Wed Oct 16 11:23:20 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/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties?rev=1532726&r1=1532725&r2=1532726&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
 Wed Oct 16 11:23:20 2013
@@ -25,8 +25,8 @@ 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.
 
 nio.eof.error=Unexpected EOF read on the socket



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to