Author: norman
Date: Tue Sep 27 10:10:09 2011
New Revision: 1176307

URL: http://svn.apache.org/viewvc?rev=1176307&view=rev
Log:
Merge ProtocolSessionFactory and Protocol interfaces

Modified:
    
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java
    
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/POP3Protocol.java
    
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3ChannelUpstreamHandler.java
    
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3Server.java
    
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPChannelUpstreamHandler.java
    
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPServer.java

Modified: 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java?rev=1176307&r1=1176306&r2=1176307&view=diff
==============================================================================
--- 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java
 (original)
+++ 
james/server/trunk/lmtpserver/src/main/java/org/apache/james/lmtpserver/netty/LMTPServer.java
 Tue Sep 27 10:10:09 2011
@@ -198,7 +198,7 @@ public class LMTPServer extends Abstract
     @Override
     protected ChannelUpstreamHandler createCoreHandler() {
         SMTPProtocol protocol = new SMTPProtocol(getProtocolHandlerChain(), 
lmtpConfig);
-        return new SMTPChannelUpstreamHandler(getProtocolHandlerChain(), 
protocol.getProtocolSessionFactory(), getLogger());
+        return new SMTPChannelUpstreamHandler(getProtocolHandlerChain(), 
protocol, getLogger());
     }
 
     @Override

Modified: 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/POP3Protocol.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/POP3Protocol.java?rev=1176307&r1=1176306&r2=1176307&view=diff
==============================================================================
--- 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/POP3Protocol.java
 (original)
+++ 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/POP3Protocol.java
 Tue Sep 27 10:10:09 2011
@@ -21,7 +21,6 @@ package org.apache.james.pop3server;
 
 import org.apache.james.protocols.api.Protocol;
 import org.apache.james.protocols.api.ProtocolSession;
-import org.apache.james.protocols.api.ProtocolSessionFactory;
 import org.apache.james.protocols.api.ProtocolTransport;
 import org.apache.james.protocols.api.handler.ProtocolHandlerChain;
 import org.slf4j.Logger;
@@ -42,15 +41,10 @@ public class POP3Protocol implements Pro
         return chain;
     }
 
+    
     @Override
-    public ProtocolSessionFactory getProtocolSessionFactory() {
-        return new ProtocolSessionFactory() {
-            
-            @Override
-            public ProtocolSession newSession(ProtocolTransport transport) {
-                return new POP3SessionImpl(logger, transport, config);
-            }
-        };
+    public ProtocolSession newSession(ProtocolTransport transport) {
+        return new POP3SessionImpl(logger, transport, config);
     }
 
     @Override

Modified: 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3ChannelUpstreamHandler.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3ChannelUpstreamHandler.java?rev=1176307&r1=1176306&r2=1176307&view=diff
==============================================================================
--- 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3ChannelUpstreamHandler.java
 (original)
+++ 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3ChannelUpstreamHandler.java
 Tue Sep 27 10:10:09 2011
@@ -22,8 +22,8 @@ package org.apache.james.pop3server.nett
 import javax.net.ssl.SSLContext;
 import javax.net.ssl.SSLEngine;
 
+import org.apache.james.protocols.api.Protocol;
 import org.apache.james.protocols.api.ProtocolSession;
-import org.apache.james.protocols.api.ProtocolSessionFactory;
 import org.apache.james.protocols.api.handler.ProtocolHandlerChain;
 import org.apache.james.protocols.impl.BasicChannelUpstreamHandler;
 import org.jboss.netty.channel.ChannelHandlerContext;
@@ -33,13 +33,13 @@ public class POP3ChannelUpstreamHandler 
 
     private boolean zeroCopy;
 
-    public POP3ChannelUpstreamHandler(ProtocolHandlerChain chain, 
ProtocolSessionFactory sessionFactory, Logger logger, boolean zeroCopy) {
-        super(chain, sessionFactory, logger);
+    public POP3ChannelUpstreamHandler(ProtocolHandlerChain chain, Protocol 
protocol, Logger logger, boolean zeroCopy) {
+        super(chain, protocol, logger);
         this.zeroCopy = zeroCopy;
     }
 
-    public POP3ChannelUpstreamHandler(ProtocolHandlerChain chain, 
ProtocolSessionFactory sessionFactory, Logger logger, SSLContext context, 
String[] enabledCipherSuites, boolean zeroCopy) {
-        super(chain, sessionFactory, logger, context, enabledCipherSuites);
+    public POP3ChannelUpstreamHandler(ProtocolHandlerChain chain, Protocol 
protocol, Logger logger, SSLContext context, String[] enabledCipherSuites, 
boolean zeroCopy) {
+        super(chain, protocol, logger, context, enabledCipherSuites);
         this.zeroCopy = zeroCopy;
     }
 
@@ -53,7 +53,7 @@ public class POP3ChannelUpstreamHandler 
             }
         }
         
-        return sessionFactory.newSession(new 
POP3ProtocolTransport(ctx.getChannel(), engine, zeroCopy));
+        return protocol.newSession(new POP3ProtocolTransport(ctx.getChannel(), 
engine, zeroCopy));
     }
 
 

Modified: 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3Server.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3Server.java?rev=1176307&r1=1176306&r2=1176307&view=diff
==============================================================================
--- 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3Server.java
 (original)
+++ 
james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/netty/POP3Server.java
 Tue Sep 27 10:10:09 2011
@@ -85,7 +85,7 @@ public class POP3Server extends Abstract
     protected void preInit() throws Exception {
         super.preInit();
         POP3Protocol protocol = new POP3Protocol(getProtocolHandlerChain(), 
theConfigData, getLogger());
-        coreHandler = new 
POP3ChannelUpstreamHandler(getProtocolHandlerChain(), 
protocol.getProtocolSessionFactory(), getLogger(), getSSLContext(), 
getEnabledCipherSuites(), false);
+        coreHandler = new 
POP3ChannelUpstreamHandler(getProtocolHandlerChain(), protocol, getLogger(), 
getSSLContext(), getEnabledCipherSuites(), false);
     }
 
 

Modified: 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPChannelUpstreamHandler.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPChannelUpstreamHandler.java?rev=1176307&r1=1176306&r2=1176307&view=diff
==============================================================================
--- 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPChannelUpstreamHandler.java
 (original)
+++ 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPChannelUpstreamHandler.java
 Tue Sep 27 10:10:09 2011
@@ -21,7 +21,7 @@ package org.apache.james.smtpserver.nett
 import javax.net.ssl.SSLContext;
 
 import org.apache.james.lifecycle.api.LifecycleUtil;
-import org.apache.james.protocols.api.ProtocolSessionFactory;
+import org.apache.james.protocols.api.Protocol;
 import org.apache.james.protocols.api.handler.ProtocolHandlerChain;
 import org.apache.james.protocols.impl.BasicChannelUpstreamHandler;
 import org.apache.james.protocols.smtp.SMTPSession;
@@ -38,12 +38,12 @@ import org.slf4j.Logger;
 public class SMTPChannelUpstreamHandler extends BasicChannelUpstreamHandler {
 
 
-    public SMTPChannelUpstreamHandler(ProtocolHandlerChain chain, 
ProtocolSessionFactory sessionFactory, Logger logger, SSLContext context, 
String[] enabledCipherSuites) {
-        super(chain, sessionFactory, logger, context, enabledCipherSuites);
+    public SMTPChannelUpstreamHandler(ProtocolHandlerChain chain, Protocol 
protocol, Logger logger, SSLContext context, String[] enabledCipherSuites) {
+        super(chain, protocol, logger, context, enabledCipherSuites);
     }
 
-    public SMTPChannelUpstreamHandler(ProtocolHandlerChain chain, 
ProtocolSessionFactory sessionFactory, Logger logger) {
-        super(chain, sessionFactory, logger);
+    public SMTPChannelUpstreamHandler(ProtocolHandlerChain chain, Protocol 
protocol, Logger logger) {
+        super(chain, protocol, logger);
     }
 
     /**

Modified: 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPServer.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPServer.java?rev=1176307&r1=1176306&r2=1176307&view=diff
==============================================================================
--- 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPServer.java
 (original)
+++ 
james/server/trunk/smtpserver/src/main/java/org/apache/james/smtpserver/netty/SMTPServer.java
 Tue Sep 27 10:10:09 2011
@@ -27,7 +27,6 @@ import org.apache.commons.configuration.
 import org.apache.james.dnsservice.api.DNSService;
 import org.apache.james.dnsservice.library.netmatcher.NetMatcher;
 import org.apache.james.protocols.api.ProtocolSession;
-import org.apache.james.protocols.api.ProtocolSessionFactory;
 import org.apache.james.protocols.api.ProtocolTransport;
 import org.apache.james.protocols.api.handler.HandlersPackage;
 import org.apache.james.protocols.impl.ResponseEncoder;
@@ -119,18 +118,12 @@ public class SMTPServer extends Abstract
         SMTPProtocol transport = new SMTPProtocol(getProtocolHandlerChain(), 
theConfigData) {
 
             @Override
-            public ProtocolSessionFactory getProtocolSessionFactory() {
-                return new ProtocolSessionFactory() {
-                    
-                    @Override
-                    public ProtocolSession newSession(ProtocolTransport 
transport) {
-                        return new ExtendedSMTPSession(theConfigData, 
getLogger(), transport);
-                    }
-                };
+            public ProtocolSession newSession(ProtocolTransport transport) {
+                return new ExtendedSMTPSession(theConfigData, getLogger(), 
transport);
             }
             
         };
-        coreHandler = new 
SMTPChannelUpstreamHandler(getProtocolHandlerChain(), 
transport.getProtocolSessionFactory(), getLogger(), getSSLContext(), 
getEnabledCipherSuites());
+        coreHandler = new 
SMTPChannelUpstreamHandler(getProtocolHandlerChain(), transport, getLogger(), 
getSSLContext(), getEnabledCipherSuites());
         
     }
 



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

Reply via email to