Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20030228222021.GA22253@openwall.com>
Date: Sat, 1 Mar 2003 01:20:21 +0300
From: Solar Designer <solar@...nwall.com>
To: popa3d-users@...ts.openwall.com
Subject: Re: Outlook 2002 duplicate emails

On Fri, Feb 28, 2003 at 03:40:06PM -0600, Thomas Hays wrote:
> Postfix behaviour does differ from Sendmail in this case. I won't even
> try to guess which is the "correct" behaviour, however, it appears
> that Postfix isn't planning on changing if I read their FAQ
> correctly:
> http://www.postfix.org/faq.html#duplicate

Well, what we need is to ensure that if duplicate messages are
produced, they are still different by at least one of the
locally-generated headers.

> Based on this, could we not tack on an extra 8 (or 16 or whatever)
> characters that would make the message unique. I have a patch that
> does this and seems to work. Basically, I modified
> pop_trans_list_or_uidl_all() and added a counter to the "for" loop and
> append that to the hash. For example, I now have hashes that look
> like:
> 80c3fb08daf1feed00000000
> 80c3fb08daf1feed00000001

This breaks another property which is even more strictly required from
UIDL: an ID for a message must not change for as long as the message
is in the mailbox.  With your example above, if one deletes the first
message and closes the POP3 session, the second message will change
its "unique" ID to that previously used by the first message.  This is
even worse than what we have currently: not only the second message's
ID not unique (it will match a previously used one), but also it has
changed across sessions (a real RFC violation)!

I am quite willing to believe that it helps with Outlook, but only
when no premature disconnects occur.

> > BTW, I have a qmail 1.03 patch which solves this by adding an
> > X-Delivery-ID: header, which popa3d is prepared to handle.  I should
> > probably make it available.
> 
> It would be great to see this patch if what I mentioned above isn't
> good.

Attached.

Please let us know if you produce a similar patch for Postfix,
preferably for the latest stable version.

-- 
/sd

View attachment "qmail-1.03-x-delivery-id.diff" of type "text/plain" (1319 bytes)

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.