darcs

Issue 2099 darcs send 2.7+ attachments are invisible in thunderbird 3.1.12

Title darcs send 2.7+ attachments are invisible in thunderbird 3.1.12
Priority bug Status resolved
Milestone 2.8.0 Resolved in 2.8.0
Superseder Nosy List galbolle, kerneis
Assigned To kerneis
Topics

Created on 2011-08-31.13:39:59 by galbolle, last changed 2011-09-06.16:04:21 by noreply.

Messages
msg14700 (view) Author: galbolle Date: 2011-08-31.13:39:58
It looks like thunderbird/icedove does not know what to do with
multipart/alternative for attachments (as opposed to body). As a result,
any mail sent with darcs 2.7 is missing its attachment when read in
thunderbird. I don't think it's technically our bug, as we're arguably
within the limits of the mime specification, but I don't think there's
much chance to get mozilla to do something about this.
msg14703 (view) Author: kerneis Date: 2011-08-31.15:48:37
On Wed, Aug 31, 2011 at 01:39:59PM +0000, Florent Becker wrote:
> It looks like thunderbird/icedove does not know what to do with
> multipart/alternative for attachments (as opposed to body). As a result,

The roundcube webmail is confused as well.  (Squirrelmail and mutt handle it
correctly, OTOH.)

> any mail sent with darcs 2.7 is missing its attachment when read in
> thunderbird. I don't think it's technically our bug, as we're arguably
> within the limits of the mime specification, but I don't think there's
> much chance to get mozilla to do something about this.

I can make a patch, but what would you suggest?  The best thing is probably
to drop the alternative part, and keep two separate attachements, giving up
semantics for the sake of interoperability.  Any other idea?

Best,
-- 
Gabriel
msg14704 (view) Author: kerneis Date: 2011-08-31.15:59:56
On Wed, Aug 31, 2011 at 04:36:07PM +0200, Florent Becker wrote:
> That was my first idea too. Otherwise, just put the patch preview into
> the body and leave the patch itself as the only attachement, which would
> prevent people from saving and trying to darcs apply the preview.

This would raise even more encoding issues (the patch encoding might be
different from the mail content).

Best,
-- 
Gabriel
msg14706 (view) Author: kerneis Date: 2011-08-31.17:28:52
Proposed patch: patch671
msg14714 (view) Author: noreply Date: 2011-09-05.20:59:21
The following patch sent by Gabriel Kerneis <kerneis@pps.jussieu.fr> updated issue issue2099 with
status=has-patch

* resolve issue2099: inline patch preview 
Ignore-this: 76a12ad951856868f9eb511163ff0582
msg14717 (view) Author: noreply Date: 2011-09-06.16:04:21
The following patch sent by Gabriel Kerneis <kerneis@pps.jussieu.fr> updated issue issue2099 with
status=resolved;resolvedin=2.8.0 HEAD

* resolve issue2099: inline patch preview 
Ignore-this: 76a12ad951856868f9eb511163ff0582
History
Date User Action Args
2011-08-31 13:39:59galbollecreate
2011-08-31 15:48:38kerneissetmessages: + msg14703
2011-08-31 15:59:57kerneissetmessages: + msg14704
2011-08-31 17:28:53kerneissetstatus: unknown -> has-patch
messages: + msg14706
2011-09-05 20:59:22noreplysetmessages: + msg14714
2011-09-06 16:04:21noreplysetstatus: has-patch -> resolved
messages: + msg14717
resolvedin: 2.8.0