Bug#684499: Streaming MOVE commands

[Emilio Jesús Gallego Arias]

Hi Adrian,

Adrian Bunk  writes:

> What was the result?
> Any chance of getting this one-line change into jessie?

So far the current Jessie packages don't include the fix, which forces
me to use my own custom packages, so indeed, this is not optimal.

jessie-backports seem to include the fix thou.

Best,
E.

Bug#684499: Streaming MOVE commands

[Adrian Bunk]

On Mon, Feb 22, 2016 at 12:36:46AM -0500, Jaldhar H. Vyas wrote:
> On Mon, 22 Feb 2016, Timo Sirainen wrote:
> 
> > On 21 Feb 2016, at 13:46, Emilio Jesús Gallego Arias  
> > wrote:
> > > 
> > > Hello Timo,
> > > 
> > > Timo Sirainen  writes:
> > > 
> > > > Thanks, looks like this was broken with Maildir and mbox formats. It
> > > > also caused expunges in some other situations to be lost. Fixed:
> > > > 
> > > > https://github.com/dovecot/core/commit/950a6e61d6c2dac961ce031bdd8b2895bc32b827
> > > 
> > > Thanks a lot for the fix, testing it now!
> > > 
> > > Is this patch suitable of being backported to 2.2.13? (Debian stable)
> > 
> > Should be.
> > 
> 
> This will definitely go into the upcoming 2.2.21 packages. (After a long
> period of stasis we're going to bring everything up to date again soon.)
> 
> I don't know if the release team will allow it for stable even though it is
> a minor change but I'll definitely bring it up for their consideration.

What was the result?

Any chance of getting this one-line change into jessie?

Thanks
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

Bug#684499: Streaming MOVE commands

[Emilio Jesús Gallego Arias]

Thanks a lot both for your help!

On Mon, 22 Feb 2016, Timo Sirainen wrote:
>> Is this patch suitable of being backported to 2.2.13? (Debian stable)
>
> Should be.
>
> BTW. This bug only meant that some expunges were ignored, which at
> worst caused unwanted email duplicates. It didn't corrupt the mailbox
> state or the client state in any way.

Thanks for the update. Testing on 2.2.13.

"Jaldhar H. Vyas"  writes:

> I don't know if the release team will allow it for stable even though
> it is a minor change but I'll definitely bring it up for their
> consideration.

Timo said the bug doesn't corrupt data, but IMVHO a case could be made
for stable-updates.

I get around 100-200 duplicated mails a day, so it really hampers the
usability of the package with my mail client (gnus).

Also, it seems that the bug be prevent people to use dsync and other
sync tools with Dovecot.

Luca, does the patch fix your dsync issues?

Unfortunately, I can't properly test the patch with 2.2.13, my mail
servers have really low mail volume to consider them "testing".

Best,
Emilio

Bug#684499: Streaming MOVE commands

[Jaldhar H. Vyas]

On Mon, 22 Feb 2016, Timo Sirainen wrote:


On 21 Feb 2016, at 13:46, Emilio Jesús Gallego Arias  
wrote:


Hello Timo,

Timo Sirainen  writes:


Thanks, looks like this was broken with Maildir and mbox formats. It
also caused expunges in some other situations to be lost. Fixed:

https://github.com/dovecot/core/commit/950a6e61d6c2dac961ce031bdd8b2895bc32b827


Thanks a lot for the fix, testing it now!

Is this patch suitable of being backported to 2.2.13? (Debian stable)


Should be.



This will definitely go into the upcoming 2.2.21 packages. (After a long 
period of stasis we're going to bring everything up to date again soon.)


I don't know if the release team will allow it for stable even though it 
is a minor change but I'll definitely bring it up for their consideration.


--
Jaldhar H. Vyas