Closed Bug 1727181 Opened 3 years ago Closed 1 year ago

Message-ID uses account domain instead of email From domain (when using Customize From Address)

Categories

(Thunderbird :: Message Compose Window, defect, P3)

Thunderbird 91

Tracking

(thunderbird_esr102 wontfix, thunderbird_esr115? fixed, thunderbird116? fixed)

RESOLVED FIXED
117 Branch
Tracking Status
thunderbird_esr102 --- wontfix
thunderbird_esr115 ? fixed
thunderbird116 ? fixed

People

(Reporter: mozbz, Assigned: babolivier, Mentored)

References

(Regressed 1 open bug)

Details

(Keywords: good-first-bug, Whiteboard: [TM:115.0.2])

Attachments

(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Firefox/91.0

Steps to reproduce:

When using the "Customize From Address" option and switching the from domain part, Thunderbird still uses the original account domain for the Message-Id calculation instead of the actual From address domain. This publicly associates addresses from different domains and uses the users' login info for message id instead of the actual email (domain) in use.

Is this a regression from a previous version?

Component: Untriaged → Message Compose Window

I don't believe so, though I admit I haven't dug into it particularly deeply.

Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Message-ID uses account domain instead of email From domain → Message-ID uses account domain instead of email From domain (when using Customize From Address)
Blocks: tb91found

We could move msgGenerateMessageId into https://searchfox.org/comm-central/source/mailnews/compose/src/MimeMessageUtils.jsm
Or rather add a copy. Once nsIMsgMdnGenerator.idl is rewritten in js the cpp msgGenerateMessageId version can go.

I want to take up on this issue. If no one else is working on it.

Great, go for it! The bug will get assigned automatically when you submit a patch.

Riya, any update?

This issue hasn't seen some activity, so I would like to attempt it if that's okay. Magnus, can I add you as reviewer when I submit a patch?

Absolutely, please see https://developer.thunderbird.net/thunderbird-development/fixing-a-bug#submitting-a-patch
Let me know if you have any questions.

Mentor: mkmelin+mozilla

Gentle ping to Magnus since the patch was uploaded to phabricator as WIP, so probably no notification was triggered by Phab.

Flags: needinfo?(mkmelin+mozilla)

Left some comments on phabricator. Luuk, please check it.

Flags: needinfo?(mkmelin+mozilla)

Luuk, are you up for finishing the patch?

Assigning this to Brendan to see if he can bring this home

Assignee: nobody → brendan
Severity: -- → S3
Priority: -- → P3
Status: NEW → ASSIGNED
Target Milestone: --- → 117 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/6244570ecfa4
Generate the Message-ID header using the From address when sending a message from the composer. r=leftmostcat

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED

Request uplift to beta when you are ready, which is a necessary step before it can be brought to version 115.
Or you can let it ride on 117 until 117 becomes beta.

Flags: needinfo?(brendan)
Attachment #9270071 - Attachment is obsolete: true

Comment on attachment 9341530 [details]
Bug 1727181 - Generate the Message-ID header using the From address when sending a message from the composer. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: Wrong usage of the message ID during compose if the user manually enters a custom "from" address.
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): Fairly low risk because the changes are self contained on an area that is well covered by tests. Requesting uplift because some interfaces are changing and it would make it difficult to uplift further fixes.

Attachment #9341530 - Flags: approval-comm-esr115?
Attachment #9341530 - Flags: approval-comm-beta?
Flags: needinfo?(brendan)
Whiteboard: [TM:115.0.2]

Comment on attachment 9341530 [details]
Bug 1727181 - Generate the Message-ID header using the From address when sending a message from the composer. r=#thunderbird-reviewers

[Triage Comment]
Approved for beta

Attachment #9341530 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9341530 [details]
Bug 1727181 - Generate the Message-ID header using the From address when sending a message from the composer. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr115

Attachment #9341530 - Flags: approval-comm-esr115? → approval-comm-esr115+
Regressions: 1846837
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: