[haiku-commits] Re: r41071 - haiku/trunk/src/servers/mail

  • From: Clemens <clemens.zeidler@xxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 22 Mar 2011 14:23:11 +1300

On Tue, 22 Mar 2011 10:47:43 +1300, Oliver Tappe <zooey@xxxxxxxxxxxxxxx> wrote: 



Hi Clemens,

On 2011-03-21 at 21:02:48 [+0100], clemens.zeidler@xxxxxxxxxxxxxx wrote:

Author: czeidler
Date: 2011-03-21 21:02:48 +0100 (Mon, 21 Mar 2011)
New Revision: 41071
Changeset: https://dev.haiku-os.org/changeset/41071

Modified:
   haiku/trunk/src/servers/mail/MailDaemon.cpp
Log:
Create an index for MAIL:account_id. If you using POP3 update to this
revision! POP3 queries for this attribute and if no mails are found with
this id it will delete the missing mails on the server (if the option is
ticked).


Are you sure this is a good idea?

If someone would download a couple of mails with Beam and then starts MDR
afterwards, all those mails would get wiped on the server, wouldn't they?
Instead of relying on a specific index, I'd rather introduce some persistent 
storage for IDs of mails that were deleted and then purge those from the
server in the next session.
POP3 stores a list of downloaded messages attached as an attribute to the mail inbox folder. When searching for gone messages this list is compared with the files in a MAIL:account_id query. MDR would download the messages again which Beam downloaded before, or does beam store the message id list in the same folder attribute? 

Regards,
        Clemens

Other related posts:

  1. Home
  2. haiku-commits
  3. Archive
  4. 03-2011