Re: missing attachment in mails

Le lundi 17 juin 2013 23:30:16, Cédric Villemain a écrit :
> 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.s
> > >> > no 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.

would you mind review this patch ?
--
Cédric Villemain +33 (0)6 20 30 22 52
http://2ndQuadrant.fr/
PostgreSQL: Support 24x7 - Développement, Expertise et Formation