[
https://issues.apache.org/jira/browse/JAMES-1869?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Tellier Benoit updated JAMES-1869:
----------------------------------
Component/s: JMAP
> JMAP protocol should limit the size of a mailbox name
> -----------------------------------------------------
>
> Key: JAMES-1869
> URL: https://issues.apache.org/jira/browse/JAMES-1869
> Project: James Server
> Issue Type: Bug
> Components: JMAP
> Reporter: Tellier Benoit
>
> When creating a mailbox with a size of more than 64K characters, cassandra
> implementation complains :
> ```
> esn_james | 13:06:06.702 [WARN ] o.e.j.s.ServletHandler - /jmap
> esn_james | com.datastax.driver.core.exceptions.InvalidQueryException:
> Index expression values may not be larger than 64K
> esn_james | at
> com.datastax.driver.core.Responses$Error.asException(Responses.java:102)
> esn_james | at
> com.datastax.driver.core.DefaultResultSetFuture.onSet(DefaultResultSetFuture.java:149)
> esn_james | at
> com.datastax.driver.core.RequestHandler.setFinalResult(RequestHandler.java:183)
> esn_james | at
> com.datastax.driver.core.RequestHandler.access$2300(RequestHandler.java:44)
> esn_james | at
> com.datastax.driver.core.RequestHandler$SpeculativeExecution.setFinalResult(RequestHandler.java:748)
> esn_james | at
> com.datastax.driver.core.RequestHandler$SpeculativeExecution.onSet(RequestHandler.java:573)
> esn_james | at
> com.datastax.driver.core.Connection$Dispatcher.channelRead0(Connection.java:1013)
> esn_james | at
> com.datastax.driver.core.Connection$Dispatcher.channelRead0(Connection.java:936)
> esn_james | at
> io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java:105)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:339)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:324)
> esn_james | at
> io.netty.handler.timeout.IdleStateHandler.channelRead(IdleStateHandler.java:254)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:339)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:324)
> esn_james | at
> io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:103)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:339)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:324)
> esn_james | at
> io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:242)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:339)
> esn_james | at
> io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:324)
> esn_james | at
> io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:847)
> esn_james | at
> io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:131)
> esn_james | at
> io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:511)
> esn_james | at
> io.netty.channel.nio.NioEventLoop.processSelectedKeysOptimized(NioEventLoop.java:468)
> esn_james | at
> io.netty.channel.nio.NioEventLoop.processSelectedKeys(NioEventLoop.java:382)
> esn_james | at
> io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:354)
> esn_james | at
> io.netty.util.concurrent.SingleThreadEventExecutor$2.run(SingleThreadEventExecutor.java:111)
> esn_james | ... 1 common frames omitted
> esn_james | Wrapped by:
> com.datastax.driver.core.exceptions.InvalidQueryException: Index expression
> values may not be larger than 64K
> esn_james | at
> com.datastax.driver.core.exceptions.InvalidQueryException.copy(InvalidQueryException.java:50)
> esn_james | at
> com.datastax.driver.core.DriverThrowables.propagateCause(DriverThrowables.java:37)
> esn_james | at
> com.datastax.driver.core.DefaultResultSetFuture.getUninterruptibly(DefaultResultSetFuture.java:214)
> esn_james | at
> com.datastax.driver.core.AbstractSession.execute(AbstractSession.java:52)
> esn_james | at
> org.apache.james.mailbox.cassandra.mail.CassandraMailboxMapper.findMailboxByPath(CassandraMailboxMapper.java:86)
> esn_james | at
> org.apache.james.mailbox.store.StoreMailboxManager.mailboxExists(StoreMailboxManager.java:607)
> esn_james | at
> org.apache.james.mailbox.store.StoreMailboxManager.createMailbox(StoreMailboxManager.java:419)
> esn_james | at
> org.apache.james.jmap.methods.SetMailboxesCreationProcessor.createMailbox(SetMailboxesCreationProcessor.java:97)
> esn_james | at
> org.apache.james.jmap.methods.SetMailboxesCreationProcessor.lambda$process$2(SetMailboxesCreationProcessor.java:75)
> esn_james | at java.util.ArrayList.forEach(ArrayList.java:1249)
> esn_james | at
> org.apache.james.jmap.methods.SetMailboxesCreationProcessor.process(SetMailboxesCreationProcessor.java:74)
> esn_james | at
> org.apache.james.jmap.methods.SetMailboxesMethod.lambda$setMailboxesResponse$0(SetMailboxesMethod.java:73)
> esn_james | at
> java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
> esn_james | at java.util.Iterator.forEachRemaining(Iterator.java:116)
> esn_james | at
> java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
> esn_james | at
> java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
> esn_james | at
> java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
> esn_james | at
> java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
> esn_james | at
> java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
> esn_james | at
> java.util.stream.ReferencePipeline.reduce(ReferencePipeline.java:484)
> esn_james | at
> org.apache.james.jmap.methods.SetMailboxesMethod.setMailboxesResponse(SetMailboxesMethod.java:74)
> esn_james | at
> org.apache.james.jmap.methods.SetMailboxesMethod.process(SetMailboxesMethod.java:66)
> esn_james | at
> org.apache.james.jmap.methods.RequestHandler.lambda$extractAndProcess$1(RequestHandler.java:67)
> esn_james | at java.util.Optional.map(Optional.java:215)
> esn_james | at
> org.apache.james.jmap.methods.RequestHandler.handle(RequestHandler.java:57)
> esn_james | at
> java.util.stream.ReferencePipeline$7$1.accept(ReferencePipeline.java:267)
> esn_james | at
> java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
> esn_james | at
> java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
> esn_james | at
> java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948)
> esn_james | at
> java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
> esn_james | at
> java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
> esn_james | at
> java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
> esn_james | at
> java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
> esn_james | at
> java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
> esn_james | at
> org.apache.james.jmap.JMAPServlet.doPost(JMAPServlet.java:71)
> esn_james | at
> javax.servlet.http.HttpServlet.service(HttpServlet.java:707)
> esn_james | at
> javax.servlet.http.HttpServlet.service(HttpServlet.java:790)
> esn_james | at
> org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:821)
> esn_james | at
> org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1685)
> esn_james | at
> org.apache.james.jmap.FirstUserConnectionFilter.doFilter(FirstUserConnectionFilter.java:71)
> esn_james | at
> org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1668)
> esn_james | at
> org.apache.james.jmap.AuthenticationFilter.doFilter(AuthenticationFilter.java:75)
> esn_james | at
> org.apache.james.jmap.BypassAuthOnRequestMethod.tryAuth(BypassAuthOnRequestMethod.java:115)
> esn_james | at
> org.apache.james.jmap.BypassAuthOnRequestMethod.doFilter(BypassAuthOnRequestMethod.java:97)
> esn_james | at
> org.apache.james.jmap.AllowAllCrossOriginRequests.doFilter(AllowAllCrossOriginRequests.java:46)
> esn_james | at
> org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1668)
> esn_james | at
> org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:581)
> esn_james | at
> org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:513)
> esn_james | at
> org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)
> esn_james | at
> org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:119)
> esn_james | at org.eclipse.jetty.server.Server.handle(Server.java:517)
> esn_james | at
> org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:306)
> esn_james | at
> org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:242)
> esn_james | at
> org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:261)
> esn_james | at
> org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:95)
> esn_james | at
> org.eclipse.jetty.io.SelectChannelEndPoint$2.run(SelectChannelEndPoint.java:75)
> esn_james | at
> org.eclipse.jetty.util.thread.strategy.ExecuteProduceConsume.produceAndRun(ExecuteProduceConsume.java:213)
> esn_james | at
> org.eclipse.jetty.util.thread.strategy.ExecuteProduceConsume.run(ExecuteProduceConsume.java:147)
> esn_james | at
> org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:654)
> esn_james | at
> org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:572)
> esn_james | at java.lang.Thread.run(Thread.java:745)
> ```
> We should have a check for this in the JMAP protocol layer. We should also
> check for this issue with IMAP. An error should be returned to the user, more
> concise and without reaching the storage layer.
> Acceptation criteria :
> - Write an integration test for JMAP
> - Write a MPT test for IMAP
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org
