Replying to mail with plaintext attachment quotes the attachment

RESOLVED FIXED

Status

MailNews Core
MIME
RESOLVED FIXED
11 years ago
9 years ago

People

(Reporter: bz, Assigned: Bienvenu)

Tracking

(Depends on: 1 bug, {fixed1.8.1.1})

Trunk
fixed1.8.1.1
Dependency tree / graph
Bug Flags:
blocking-thunderbird2 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

STEPS TO REPRODUCE:

1)  Create a 600KB text file (like a profile, say).
2)  Send it as an attachment to an email.
3)  Read this email in mailnews.
4)  Reply to this email.

EXPECTED RESULTS:  Mailnews quotes the actual email text, but NOT the huge attachment.

ACTUAL RESULTS:  The CPU is at 100% and the app is locked up for a minute or two as we quote the whole 600KB of attachment.
Oh, you need to have quoting of replies enabled, of course.
Depends on: 358275
(Assignee)

Comment 2

11 years ago
yeah, I've always hated this, and we've always done it, for as long as I can remember. The idea is probably that since we display text attachments, we should quote them. Especially in the case where the message body is just a txt attachment. I'd be happy to get rid of this behavior, personally. The change would be in libmime, I believe.
Component: MailNews: Composition → MailNews: MIME
I can see an argument for it if the message body is just the txt attachment, yeah.  But in general, it seems kind of odd...
(Assignee)

Comment 4

11 years ago
This turns out to be non-trivial because of the way libmime works. I think the place to fix this is in MimeInlineTextPlain_parse_eof - check if we're quoting, and don't output any text in the case where the part is not the message body. But figuring that out might be tricky, because we have to handle the various mime multipart cases. One possibility is to check if the part has a filename and isn't a top-level part.
(Assignee)

Comment 5

11 years ago
Created attachment 243835 [details] [diff] [review]
proposed fix

this works for me...maybe I'm particularly sensitive to this bug, because I often get sent imap protocol logs, and replying to those messages is a big pain, but I think the benefits outweigh the risk of occasionally not quoting something the user might want quoted.
Assignee: nobody → bienvenu
Status: NEW → ASSIGNED
Attachment #243835 - Flags: superreview?(mscott)

Comment 6

11 years ago
Comment on attachment 243835 [details] [diff] [review]
proposed fix

I'm on board with trying this new behavior out.
Attachment #243835 - Flags: superreview?(mscott) → superreview+
(Assignee)

Comment 7

11 years ago
fixed on trunk - nominating for tb 2, to keep on radar, but will let bake on trunk for a while.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Flags: blocking-thunderbird2?
Resolution: --- → FIXED

Comment 8

11 years ago
Could this be behind bug 358938?  No particular reason for me to assume so other than my attempts at testing this fix kept putting the crash in my face.
(Assignee)

Comment 9

11 years ago
I doubt it - that crash has been around for weeks, or at least one that looks similar...

Updated

11 years ago
OS: Linux → All
Hardware: PC → All

Comment 10

11 years ago
OK, I've turned off inline-spellchecking and so I'm not suffering from the crash in the trunk anymore.  I can verify this is fixed (TB 3a1-1031, Win2K), but bug 161775 still remains open for all the other displayed-inline attachment types.  I won't mark the bug Verified, pending the TB2.0 decision.

Comment 11

11 years ago
This bug should apply not only to txt attachments but to images also? Currently image attachments too get quoted in a reply.
I guess the proposed fix works anyway for all mimetype not just txt?

Comment 12

11 years ago
Comment on attachment 243835 [details] [diff] [review]
proposed fix

David, should we take this on the branch?
Attachment #243835 - Flags: approval-thunderbird2?

Updated

11 years ago
Flags: blocking-thunderbird2? → blocking-thunderbird2+
(Assignee)

Comment 13

11 years ago
I haven't heard any complaints on the trunk - so yeah, we should try this out on the branch.
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.1.1

Comment 14

11 years ago
Comment on attachment 243835 [details] [diff] [review]
proposed fix

I've been enjoying this behavior on the trunk.
Attachment #243835 - Flags: approval-thunderbird2? → approval-thunderbird2+
(Assignee)

Comment 15

11 years ago
I already landed this, after beta 1.

Comment 16

10 years ago
Would it be possible to add an option to control this behavior?  I may be in the minority, but I actually liked the old behavior.

Updated

10 years ago
Depends on: 384599

Comment 17

10 years ago
I have to agree with Mr Lustbader, I miss this behaviour, it would be nice to have some about:config goodness to restore the original behaviour.

Comment 18

10 years ago
I'd like to voice my support for making this a configurable feature. I handle support tickets which our tracking system escalates to us by email with the ticket information as a text attachment. Since upgrading to 2.0, replying to the tickets doesn't automatically carry the ticket information anymore. I'd really like to have the option of quoting the attachment in my reply.

Comment 19

10 years ago
There is an enhancement request (bug 384599) pending to introduce this option.
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.