969358 - Reply-All on an email I previously sent To:myself with Bcc:others has no recipients

Status: RESOLVED FIXED

(MailNews Core :: Composition, defect)

Description

[Ed Morton]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0 (Beta/Release)
Build ID: 20131205075310

Steps to reproduce:

Sent an email to myself and blind-copied several other people. That was fine. Later tried to use Reply-All on the email in my Sent folder to send a followup to the original mailing list.


Actual results:

Neither the To nor the Bcc fields were populated. 


Expected results:

The To and Bcc fields should have been populated same as in the original email. This is a brand new bug in the Thunderbird release that got automatically installed yesterday, 24.3.0.

Comment 1

[Chris]

Thx Ed.

I can confirm this is exactly what happens. I tried to read the updates in the latest version of TB which changed how reply-to worked (as there was apparently a bug connected with handling alternate reply-to addresses), but I could not work out if stopping reply-to on bcc: email lists was an intentional change or not.  

I can say that reply-all on emails with bcc lists in, say, gmail are not processed as TB did prior to this latest release .... which is a shame, as it was the last reason for me to stick with TB rather than setting-up a newsgroup and migrating away.

If someone could confirm what the product is meant to do ... it would be nice, then we can decide.

Many thx.

Comment 2

[rsx11m]

Magnus, this is not bug 968270. Another possible fall-out from bug 933555?

Comment 3

[rsx11m]

On the other hand, since both of you have Gmail accounts, the Bcc headers aren't retained in the "Sent Mail" folder, thus in general bug 968270 comment #4 applies. However,

> Neither the To nor the Bcc fields were populated.

The "To" headers still populate for me when trying this with my Gmail account and auto-bcc set.

Comment 4

[rsx11m]

Various reports on GetSatisfaction that Bcc address were filled in a reply-all-from-own-message situation in prior versions but no longer with 24.3.0, apparently not being auto-bcc cases.

http://gsfn.us/t/4fndb
http://gsfn.us/t/4frdg
http://gsfn.us/t/4fqzr
http://gsfn.us/t/4flot

Comment 5

[Ed Morton]

FWIW since gmail seems to be some kind of issue, I only use this gmail account for posting to usenet and forums. The problem I'm reporting happens with my non-gmail personal and work email accounts.

Comment 6

[Magnus Melin [:mkmelin]]

Yeah this should have the same fix as bug 968270.

Comment 7

[Tony Mechelynck [:tonymec]]

In reply to comment #0:

Workaround: Instead of "Reply to all", try "Message → Edit Message as New".

Comment 8

[Magnus Melin [:mkmelin]]

This needs another fix than bug 968270 after all.

Comment 9

[Chris]

(In reply to Tony Mechelynck [:tonymec] from comment #7)
> In reply to comment #0:
> 
> Workaround: Instead of "Reply to all", try "Message → Edit Message as New".

       Many thanks for the workaround.  My mistake for not investigating the options more thoroughly, but would be nice to get Reply-all working again at some point.

Comment 10

[Magnus Melin [:mkmelin]]

Attachment: proposed fix

Should be applied on top of bug 968270.

Comment 11

[Magnus Melin [:mkmelin]]

Comment on attachment 8375742 [details] [diff] [review]
proposed fix

Review of attachment 8375742 [details] [diff] [review]:
-----------------------------------------------------------------

(should be applied on top of the patch for bug 968270)

Comment 12

[Frank Butterfield]

This doesn't only affect Gmail accounts.  This is happening with all IMAP accounts I connect to via Thunderbird.  Additionally I believe that the issue is that some limit was introduced in this release that only allows so many characters or bytes to be passed when hitting reply to all.  I've noticed that if the To, CC, and BCC fields are short enough when combined together, all will pass to the new message.  I haven't been able to test this, but did want to share what I've seen anecdotally.

Comment 13

[:Irving Reid (No longer working on Firefox)]

Comment on attachment 8375742 [details] [diff] [review]
proposed fix

Review of attachment 8375742 [details] [diff] [review]:
-----------------------------------------------------------------

I still wish I knew where we introduced the regression that caused bug 933555 in the first place; it seems to me like all of these problems are falling out from that patch. I'm torn here - I can r+ this patch, because it solves the immediate problem in this bug, but I still feel like we're sticking bandages on a deeper problem. Mark, do you have any time to look at this?

::: mailnews/compose/src/nsMsgCompose.cpp
@@ +2559,5 @@
> +                break;
> +              }
> +              else if (ccEmailAddresses.Contains(curIdentityEmail2))
> +              {
> +                // If you auto-Cc yourself your email would be in Cc - but we 

Trailing white space

Comment 14

[Magnus Melin [:mkmelin]]

(In reply to :Irving Reid from comment #13)
> I still wish I knew where we introduced the regression that caused bug
> 933555 in the first place;

That would be the (lookupIdentity == lookupIdentity2) check of bug 917231. Basically, fixing one case has regressed another unexpected case.

Comment 15

[Magnus Melin [:mkmelin]]

https://hg.mozilla.org/comm-central/rev/f8ffe310f773 -> FIXED

Comment 17

[Magnus Melin [:mkmelin]]

I'd encourage people to try out nightly builds with the fix included - you can get one from here: http://ftp.mozilla.org/pub/mozilla.org/thunderbird/nightly/latest-comm-central/

Comment 18

[rsx11m]

Comment on attachment 8375742 [details] [diff] [review]
proposed fix

Bug 933555 also landed on comm-aurora at that time, thus SeaMonkey will need this patch on comm-beta to be effective for SM 2.25.

Adding this to the existing branch requests.

Comment 19

[Mark Banner (:standard8)]

https://hg.mozilla.org/releases/comm-aurora/rev/fa1d2b999151

Comment 20

[Mark Banner (:standard8)]

Comment on attachment 8375742 [details] [diff] [review]
proposed fix

Unfortunately this doesn't apply cleanly to beta. Magnus, could you upload a patch that works for Beta & ESR?

Thanks

Comment 21

[Mark Banner (:standard8)]

Comment on attachment 8375742 [details] [diff] [review]
proposed fix

I've fixed the merge issue and have been able to land this bug here:

https://hg.mozilla.org/releases/comm-beta/rev/a5d95c28e762

Comment 22

[Mark Banner (:standard8)]

(In reply to Mark Banner (:standard8) from comment #21)
> Comment on attachment 8375742 [details] [diff] [review]
> proposed fix
> 
> I've fixed the merge issue and have been able to land this bug here:
> 
> https://hg.mozilla.org/releases/comm-beta/rev/a5d95c28e762

Plus a bustage-fix follow-up: https://hg.mozilla.org/releases/comm-beta/rev/f77732d5473c

Comment 23

[Mark Banner (:standard8)]

https://hg.mozilla.org/releases/comm-esr24/rev/5d0f4ea39b13
https://hg.mozilla.org/releases/comm-esr24/rev/50a3490ab365

Comment 24

[Paul H]

Hi,

This bug seems to have reappeared. I've updated to the latest release (31.2.0) and it's doing excatly as described in the bug report. Create an email to yourself, BCC some other addresses (many or just one), send then go into Sent, hit Reply All and all address fields (To and BCC) are blank.

Do I need to post this as a new bug or can this be reopened? (Sorry, I'm new).

Comment 25

[Magnus Melin [:mkmelin]]

This is closed, so open a new bug. (I do think it's highly likely not to be a bug in thunderbird, do make sure to test with "thunderbird.exe -safe-mode"