[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [sup-devel] [Heliotrope/Turnsole] How to use IMAP?



Excerpts from Michael Stapelberg's message of 2012-02-22 13:39:42 -0800:
> Thanks for adding code for this. I got the chance to test this today,
> and I’m afraid this doesn’t seem to be working as I would expect.
> 
> As far as I can tell from glancing at the source and the state file, this code
> examines the date stored in every mail, sorts that, then checks where it
> stopped before, and continues from there. I think this is a horrible way of
> doing things for two reasons:
> 
> 1) It involves opening every single email on disk.
> 2) Date specifications in emails are not to be trusted (think spam).

My understanding of the Maildir specification is that there is no way to
determine the order of messages besides reading the date headers.
Ordering messages correctly at import time is important because
Heliotrope only serves messages in the reverse order to which they were
imported. If there is a better solution, please feel free to educate me.

Is there any problem with the current code besides the fact that you
don't like the big scan?

> Note that adding mails via stdin doesn’t work until you change the code like this:
> https://github.com/wmorgan/heliotrope/pull/34

Merged, thank you.

-- 
William <wmorgan@masanjin.net>
_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel