james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tellier Benoit (JIRA)" <server-...@james.apache.org>
Subject [jira] [Updated] (JAMES-1869) JMAP protocol should limit the size of a mailbox name
Date Fri, 25 Nov 2016 05:54:58 GMT

     [ 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-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Mime
View raw message