Message-ID uses account domain instead of email From domain (when using Customize From Address)
Categories
(Thunderbird :: Message Compose Window, defect, P3)
Tracking
(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)
(deleted),
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
wsmwk
:
approval-comm-esr115+
|
Details |
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.
Comment 1•3 years ago
|
||
Is this a regression from a previous version?
I don't believe so, though I admit I haven't dug into it particularly deeply.
Comment 3•3 years ago
|
||
https://searchfox.org/comm-central/rev/c4f87ae30bf85e5a2654b2d875b387d100aab33d/mailnews/compose/src/SmtpService.jsm#111-114
I suppose we should move the message id generation into js around there.
Comment 4•3 years ago
|
||
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.
Updated•3 years ago
|
Comment 6•3 years ago
|
||
Great, go for it! The bug will get assigned automatically when you submit a patch.
Comment 7•3 years ago
|
||
Riya, any update?
Comment 8•3 years ago
|
||
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?
Comment 9•3 years ago
|
||
Absolutely, please see https://developer.thunderbird.net/thunderbird-development/fixing-a-bug#submitting-a-patch
Let me know if you have any questions.
Comment 10•3 years ago
|
||
Comment 11•2 years ago
|
||
Gentle ping to Magnus since the patch was uploaded to phabricator as WIP, so probably no notification was triggered by Phab.
Comment 12•2 years ago
|
||
Left some comments on phabricator. Luuk, please check it.
Comment 13•1 years ago
|
||
Luuk, are you up for finishing the patch?
Comment 14•1 year ago
|
||
Assigning this to Brendan to see if he can bring this home
Assignee | ||
Comment 15•1 year ago
|
||
Loosely inspired by the patch from @lkkmpn.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Updated•1 year ago
|
Comment 16•1 year ago
|
||
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
Comment 17•1 year ago
|
||
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.
Updated•1 year ago
|
Updated•1 year ago
|
Comment 18•1 year ago
|
||
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.
Updated•1 year ago
|
Updated•1 year ago
|
Comment 19•1 year ago
|
||
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
Comment 20•1 year ago
|
||
bugherder uplift |
Thunderbird 116.0b5:
https://hg.mozilla.org/releases/comm-beta/rev/71b19af1b9d6
Comment 21•1 year ago
|
||
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
Comment 22•1 year ago
|
||
bugherder uplift |
Thunderbird 115.1.0:
https://hg.mozilla.org/releases/comm-esr115/rev/b7249e50ac7b
Description
•