Re: missing attachment in mails

From: Cédric Villemain <cedric(at)2ndquadrant(dot)com>
To: pgsql-www(at)postgresql(dot)org
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Stephen Frost <sfrost(at)snowman(dot)net>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Subject: Re: missing attachment in mails
Date: 2013-06-17 21:30:16
Message-ID: 201306172330.22018.cedric@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

Le lundi 17 juin 2013 15:56:44, Magnus Hagander a écrit :
> On Mon, Jun 17, 2013 at 11:39 AM, Cédric Villemain
>
> <cedric(at)2ndquadrant(dot)com> wrote:
> > Le mercredi 29 mai 2013 19:36:46, Stephen Frost a écrit :
> >> * Cédric Villemain (cedric(at)2ndquadrant(dot)com) wrote:
> >> > I've search another example and found one from Stephen Frost (Sign +
> >> > attachment):
> >> >
> >> > http://www.postgresql.org/message-id/20130413030619.GN4361@tamriel.sno
> >> > wma n.net
> >> >
> >> > it contains an attachment not visible in the archive.
> >>
> >> Huh, neat. Yeah, this is the way mutt handles attachments, etc, and is
> >> pretty common (at least, among open source folks :). We should probably
> >> look into figuring out how to support it..
> >
> > I had no time to have a look at that.
> > CF is on its way but the mails linked from CF items don't contain
> > attachment...
> >
> > How can I help fixing that ?
>
> A good start would be to identify exactly when this happens. Not based
> on which MUA is used, but based on which types of messages. The first
> one on your list (the only one i have time to check right this moment
> - boarding a flight) is a multipart/signed containing a
> multipart/mixed for example - might it be that the problem lies in
> nested multipart or so?

Correct, issue was that recursion entered only multipart/mixed, I've added
multipart/signed. See patch 0001.
I've also attached a patch to correct a typo.
--
Cédric Villemain +33 (0)6 20 30 22 52
http://2ndQuadrant.fr/
PostgreSQL: Support 24x7 - Développement, Expertise et Formation

Attachment Content-Type Size
0001-Allows-parser-to-recurse-in-multipart-signed.patch text/x-patch 1.1 KB
0002-Correct-a-typo.patch text/x-patch 1.1 KB

In response to

Responses

Browse pgsql-www by date

  From Date Subject
Next Message Alvaro Herrera 2013-06-17 21:35:50 Re: Threadview for messages: really cool but big flaw
Previous Message Jeff Janes 2013-06-17 21:16:47 Re: Threadview for messages: really cool but big flaw