cayenne-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From amania...@apache.org
Subject svn commit: r591892 - /cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/remote/service/BaseRemoteService.java
Date Mon, 05 Nov 2007 06:30:43 GMT
Author: amaniatis
Date: Sun Nov  4 22:30:42 2007
New Revision: 591892

URL: http://svn.apache.org/viewvc?rev=591892&view=rev
Log:
Marcin Skladaniec contributed this fix. His problems with errors not being propagated properly
from server to client were largely because of this missing flush().

Without it, the original cause message was being lost. We might still need to consider a switch
to turn this off/on in the client for security reasons, but for now this reinstate the original
purpose of the code.


Modified:
    cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/remote/service/BaseRemoteService.java

Modified: cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/remote/service/BaseRemoteService.java
URL: http://svn.apache.org/viewvc/cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/remote/service/BaseRemoteService.java?rev=591892&r1=591891&r2=591892&view=diff
==============================================================================
--- cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/remote/service/BaseRemoteService.java
(original)
+++ cayenne/main/trunk/framework/cayenne-jdk1.4-unpublished/src/main/java/org/apache/cayenne/remote/service/BaseRemoteService.java
Sun Nov  4 22:30:42 2007
@@ -168,6 +168,7 @@
             }
 
             th.printStackTrace(causeMessage);
+            causeMessage.flush();
             Exception cause = new Exception(causeMessageStream.toString());
             
             StringBuffer wrapperMessage = new StringBuffer();



Mime
View raw message