You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2012/01/11 15:04:02 UTC

svn commit: r1230047 - in /james/protocols/trunk: api/src/main/java/org/apache/james/protocols/api/ api/src/main/java/org/apache/james/protocols/api/future/ api/src/main/java/org/apache/james/protocols/api/handler/ netty/src/main/java/org/apache/james/...

Author: norman
Date: Wed Jan 11 14:04:02 2012
New Revision: 1230047

URL: http://svn.apache.org/viewvc?rev=1230047&view=rev
Log:
Cleanup and javadocs

Modified:
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfiguration.java
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfigurationImpl.java
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolServer.java
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStartTlsResponse.java
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStreamResponseImpl.java
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/LifecycleAwareProtocolHandler.java
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/ProtocolHandlerResultHandler.java
    james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/UnknownCommandHandler.java
    james/protocols/trunk/netty/src/main/java/org/apache/james/protocols/netty/HandlerConstants.java

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfiguration.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfiguration.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfiguration.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfiguration.java Wed Jan 11 14:04:02 2012
@@ -19,6 +19,11 @@
 
 package org.apache.james.protocols.api;
 
+/**
+ * Configuration which is used in scope of a Protocol
+ * 
+ *
+ */
 public interface ProtocolConfiguration {
 
     

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfigurationImpl.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfigurationImpl.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfigurationImpl.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolConfigurationImpl.java Wed Jan 11 14:04:02 2012
@@ -20,7 +20,15 @@
 package org.apache.james.protocols.api;
 
 
+/**
+ * Default implementation of a {@link ProtocolConfiguration} which allows to easily set the different configurations. 
+ * 
+ * Be aware that set and get operations are not thread-safe
+ * 
+ *
+ */
 public class ProtocolConfigurationImpl implements ProtocolConfiguration {
+    
     private String greeting;
     private String softwareName = "JAMES Protocols Server";
     private String helloName = "localhost";

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolServer.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolServer.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolServer.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/ProtocolServer.java Wed Jan 11 14:04:02 2012
@@ -21,6 +21,10 @@ package org.apache.james.protocols.api;
 import java.net.InetSocketAddress;
 import java.util.List;
 
+/**
+ * A {@link ProtocolServer} accept inbound traffic and handle it. Basically the protocols API can be used to handle every "line based" protocol in an easy fashion
+ *
+ */
 public interface ProtocolServer {
 
     /**

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStartTlsResponse.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStartTlsResponse.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStartTlsResponse.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStartTlsResponse.java Wed Jan 11 14:04:02 2012
@@ -22,6 +22,11 @@ package org.apache.james.protocols.api.f
 import org.apache.james.protocols.api.Response;
 import org.apache.james.protocols.api.StartTlsResponse;
 
+/**
+ * Special {@link FutureResponse} which will get notified once a {@link StartTlsResponse} is ready
+ * 
+ *
+ */
 public class FutureStartTlsResponse extends FutureResponseImpl implements StartTlsResponse{
 
 

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStreamResponseImpl.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStreamResponseImpl.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStreamResponseImpl.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/future/FutureStreamResponseImpl.java Wed Jan 11 14:04:02 2012
@@ -24,6 +24,11 @@ import java.io.InputStream;
 import org.apache.james.protocols.api.Response;
 import org.apache.james.protocols.api.StreamResponse;
 
+/**
+ * Special {@link FutureResponse} which wraps a {@link StreamResponse} and so provide an async way to get notified about ready responses
+ * 
+ *
+ */
 public class FutureStreamResponseImpl extends FutureResponseImpl implements StreamResponse{
 
     /**

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/LifecycleAwareProtocolHandler.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/LifecycleAwareProtocolHandler.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/LifecycleAwareProtocolHandler.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/LifecycleAwareProtocolHandler.java Wed Jan 11 14:04:02 2012
@@ -28,7 +28,7 @@ public interface LifecycleAwareProtocolH
     /**
      * Destroy object
      */
-    public void destroy();
+    void destroy();
 
 
 }

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/ProtocolHandlerResultHandler.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/ProtocolHandlerResultHandler.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/ProtocolHandlerResultHandler.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/ProtocolHandlerResultHandler.java Wed Jan 11 14:04:02 2012
@@ -22,6 +22,15 @@ package org.apache.james.protocols.api.h
 import org.apache.james.protocols.api.ProtocolSession;
 import org.apache.james.protocols.api.Response;
 
+/**
+ * An special {@link ProtocolHandler} which allows to <strong>intercept</code> returned {@link Response}'s and act on them.
+ * 
+ * This could be to gather statistics or even replace them. 
+ * @author Maurer
+ *
+ * @param <R>
+ * @param <S>
+ */
 public interface ProtocolHandlerResultHandler<R extends Response, S extends ProtocolSession> extends ProtocolHandler{
 
     /**

Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/UnknownCommandHandler.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/UnknownCommandHandler.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/UnknownCommandHandler.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/handler/UnknownCommandHandler.java Wed Jan 11 14:04:02 2012
@@ -24,6 +24,13 @@ import java.util.Collections;
 
 import org.apache.james.protocols.api.ProtocolSession;
 
+/**
+ * A special {@link CommandHandler} implementation which should be extended by {@link CommandHandler}'s which should get called for unknown command. So this is some kind
+ * of a <strong>fallback</strong> {@link CommandHandler} which will get executed if no other matching {@link CommandHandler} could be found for a given command.
+ * 
+ *
+ * @param <S>
+ */
 public abstract class UnknownCommandHandler<S extends ProtocolSession> implements CommandHandler<S>{
 
     /**

Modified: james/protocols/trunk/netty/src/main/java/org/apache/james/protocols/netty/HandlerConstants.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/netty/src/main/java/org/apache/james/protocols/netty/HandlerConstants.java?rev=1230047&r1=1230046&r2=1230047&view=diff
==============================================================================
--- james/protocols/trunk/netty/src/main/java/org/apache/james/protocols/netty/HandlerConstants.java (original)
+++ james/protocols/trunk/netty/src/main/java/org/apache/james/protocols/netty/HandlerConstants.java Wed Jan 11 14:04:02 2012
@@ -22,29 +22,29 @@ import org.jboss.netty.channel.ChannelHa
 import org.jboss.netty.channel.ChannelPipeline;
 
 /**
- * Provide the keys under which the {@link ChannelHandler}'s are stored in the {@link ChannelPipeline}
+ * Provide the keys under which the {@link ChannelHandler}'s are stored in the
+ * {@link ChannelPipeline}
+ * 
  * 
- *
  */
 public interface HandlerConstants {
 
-	public static final String SSL_HANDLER = "sslHandler";
-	
-	public static final String GROUP_HANDLER ="groupHandler";
-	
-	public static final String CONNECTION_LIMIT_HANDLER =" connectionLimit";
-	
-	public static final String CONNECTION_PER_IP_LIMIT_HANDLER ="connectionPerIpLimit";
-	
-	public static final String FRAMER = "framer";
-	
-	public static final String EXECUTION_HANDLER = "executionHandler";
-	
-	public static final String TIMEOUT_HANDLER = "timeoutHandler";
-	
-	public static final String CORE_HANDLER = "coreHandler";
+    public static final String SSL_HANDLER = "sslHandler";
+
+    public static final String GROUP_HANDLER = "groupHandler";
+
+    public static final String CONNECTION_LIMIT_HANDLER = " connectionLimit";
+
+    public static final String CONNECTION_PER_IP_LIMIT_HANDLER = "connectionPerIpLimit";
+
+    public static final String FRAMER = "framer";
+
+    public static final String EXECUTION_HANDLER = "executionHandler";
+
+    public static final String TIMEOUT_HANDLER = "timeoutHandler";
 
-	public static final String CHUNK_HANDLER = "chunkHandler";
+    public static final String CORE_HANDLER = "coreHandler";
 
+    public static final String CHUNK_HANDLER = "chunkHandler";
 
 }



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