ff-wl commented on a change in pull request #886:
URL: https://github.com/apache/james-project/pull/886#discussion_r808755975



##########
File path: 
server/protocols/protocols-managesieve/src/main/java/org/apache/james/managesieveserver/netty/ManageSieveServer.java
##########
@@ -78,22 +77,27 @@ protected String getDefaultJMXName() {
     }
 
     @Override
-    protected ChannelUpstreamHandler createCoreHandler() {
+    protected ChannelInboundHandlerAdapter createCoreHandler() {
         return new ManageSieveChannelUpstreamHandler(manageSieveProcessor,
             getEncryption(),
             LOGGER);
     }
 
     @Override
-    protected ChannelPipelineFactory createPipelineFactory(final ChannelGroup 
group) {
+    protected AbstractChannelPipelineFactory createPipelineFactory(final 
ChannelGroup group) {
+
+        return new AbstractChannelPipelineFactory(0,0, 0, group, 
createFrameHandlerFactory()) {

Review comment:
       added a new constructor to AbstractChannelPipelineFactory without 
timeout and rate limit options (corresponding handlers are disabled)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



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

Reply via email to