Cyrus 2.3.16 \Seen flag problem

2011-04-21 Thread Julien Coloos
Hi, We encountered a bug in cyrus 2.3.16: mails freshly delivered by LMTP were already flagged \Seen while not having been read by anyone. After some investigations, we tracked it down to the code in charge of updating the seen db: function index_checkseen in imap/index.c file. That complicated co

Re: Cyrus 2.3.16 \Seen flag problem

2011-04-21 Thread Bron Gondwana
On Thu, Apr 21, 2011 at 06:46:25PM +0200, Julien Coloos wrote: > It may be possible that some other set of conditions would trigger a > similar issue. > We tried to think of a clean way to solve it, but provided the > complexity of that code we gave up (mainly fearing nasty side effetcs > and intro

Re: Cyrus 2.3.16 \Seen flag problem

2011-04-22 Thread Julien Coloos
Le 21/04/2011 20:40, Bron Gondwana a écrit : Unfortunately, moving to 2.4.x version is not yet an option for us :( Why not? So ... is there some cyrus guru expert out there with enough knowledge of that part of the code to fix it in the 2.3.x version ? Yes - I already did once... and I'd ra

Michel's Proprietary Patches [Was: Cyrus 2.3.16 \Seen flag problem]

2011-04-22 Thread Adam Tauno Williams
On Fri, 2011-04-22 at 15:38 +0200, Julien Coloos wrote: > However we think that a better way would be to split at attachment > level instead of body level, it could be possible to save base64 > overhead, permit de-duping and do data compression (i.e on PDF or > Office). May be it would be nice to a

Re: Michel's Proprietary Patches [Was: Cyrus 2.3.16 \Seen flag problem]

2011-06-09 Thread Sébastien Michel
2011/4/22 Adam Tauno Williams > On Fri, 2011-04-22 at 15:38 +0200, Julien Coloos wrote: > > However we think that a better way would be to split at attachment > > level instead of body level, it could be possible to save base64 > > overhead, permit de-duping and do data compression (i.e on PDF or