Closed Bug 543701 Opened 10 years ago Closed 9 years ago
Email address of secondary identities not recognized as "You" address book contact
User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.2) Gecko/20100115 Epiphany/2.22 Firefox/3.6 Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:18.104.22.168) Gecko/20100127 Lightning/1.0b2pre Thunderbird/3.0.1 If you add an identity (with a different mail address that the main one) mails received for that mail address are not recognized as 'to "You"' is this expected? Reproducible: Always Steps to Reproduce: 1.Create a mail account that receive mails for 2 or more mail address 2.create an identity for the second mail address 3.receive an email for the second mail address Actual Results: The "to" address on the message header is not marked as "You" Expected Results: The "to" address on the message header is marked as "You"
I see this too, confirming it. However, fixing this would cause a lot of confusion without the fix from bug 478466.
Status: UNCONFIRMED → NEW
Ever confirmed: true
I've looked at this in a little more detail, and it only seems to happen when you have two identities whose domain names are the same.
Comment 2 may be inaccurate. The code for checking identities was pretty strange. The attached patch is much simpler and should resolve the issue. No tests yet, though.
Assignee: nobody → squibblyflabbetydoo
Status: NEW → ASSIGNED
Can anyone who has experienced this tell me what identities they have in Thunderbird? (You can anonymize the addresses of course.) I'd really like to get this fixed, but I'm sure it'll require a test to get checked in, and I don't really know what to test for.
Comment on attachment 471421 [details] [diff] [review] Simple patch, no tests yet Blake, could you take a look at this? I think this change is sane, but I'm at a loss as to how to test it, and how big a test I really need...
Attachment #471421 - Flags: feedback?(bwinton)
Comment on attachment 471421 [details] [diff] [review] Simple patch, no tests yet Couldn't we get essentially the same behaviour with a much smaller change by passing aEmailAddress in to getBestIdentity as the second "optionalHint" parameter? Thanks, Blake.
Attachment #471421 - Flags: feedback?(bwinton) → feedback-
(P.S. Sorry about the delay in getting back to you on this. I'll try to do better in the future.)
I think that would work, but it seems a little roundabout, and in the back of my head, I worry about the performance, since getBestIdentity does roughly twice as much work as IsIdentity. Maybe that's not a big deal in practice, though. (Of course, it might become a bigger deal after bug 591730 and/or bug 312821 are fixed.)
This is about as simple as it gets. It probably works (I never did quite figure out a comprehensive list of causes for this problem), and it'll get tested by way of bug 527550. Much as I'd like an IsIdentity function (it just seems like a sensible thing), it's probably not a big enough deal to worry about, since at best, it would only get used here and in some logic for Reply All.
Attachment #471421 - Attachment is obsolete: true
Comment on attachment 483612 [details] [diff] [review] The simplest possible patch (Assigning to me for review, cause otherwise I'll never get around to reviewing it.)
Attachment #483612 - Flags: review?(bwinton)
Comment on attachment 483612 [details] [diff] [review] The simplest possible patch Apparently it'll take me ages to get around to reviewing it even if I _do_ ask myself for review. :P Anyways, I like it. It seems simple, and reasonably correct. Thanks, Blake.
Attachment #483612 - Flags: review?(bwinton) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.