Author: norman
Date: Wed Jun 22 18:38:39 2011
New Revision: 1138574

URL: http://svn.apache.org/viewvc?rev=1138574&view=rev
Log:
Fix problem with PIPELINE support when using NIO version of SMTPServer. See 
JAMES-1277

Modified:
    
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPNettySession.java

Modified: 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPNettySession.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPNettySession.java?rev=1138574&r1=1138573&r2=1138574&view=diff
==============================================================================
--- 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPNettySession.java
 (original)
+++ 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPNettySession.java
 Wed Jun 22 18:38:39 2011
@@ -116,8 +116,11 @@ public class SMTPNettySession extends Ab
      */
     public void pushLineHandler(LineHandler<SMTPSession> 
overrideCommandHandler) {
         lineHandlerCount++;
-
-        getChannelHandlerContext().getPipeline().addAfter("timeoutHandler", 
"lineHandler" + lineHandlerCount, new 
LineHandlerUpstreamHandler<SMTPSession>(overrideCommandHandler));
+        // Add the linehandler in front of the coreHandler so we can be sure 
+        // it is executed with the same ExecutorHandler as the coreHandler (if 
one exist)
+        // 
+        // See JAMES-1277
+        getChannelHandlerContext().getPipeline().addBefore("coreHandler", 
"lineHandler" + lineHandlerCount, new 
LineHandlerUpstreamHandler<SMTPSession>(overrideCommandHandler));
     }
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to