Closed Bug 1993555 Opened 5 months ago Closed 5 months ago

[ews] ReplyTo can only contain a single recipient but should be able to contain multiple.

Categories

(MailNews Core :: Networking: Exchange, defect)

defect

Tracking

(thunderbird145 fixed)

RESOLVED FIXED
146 Branch
Tracking Status
thunderbird145 --- fixed

People

(Reporter: edicharry, Assigned: edicharry)

References

Details

Attachments

(1 file)

We saw an example in the wild of email messages being returned with multiple recipients in the ReplyTo field, but ews-rs is only allowing a single value. The documentation for that field of an Item states "Identifies a set of addresses to which replies should be sent.", so I think we're in the wrong here. I'll verify the multiplicity of the other fields while I'm fixing this.

The symptom of this issue appears to be failing to download messages, which is something we've seen a smattering of reports for.

See Also: → 1980463

Pushed by kaie@kuix.de:
https://hg.mozilla.org/comm-central/rev/efdfb2936679
Allow multiple reply-to addresses on emails. r=babolivier

Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Pushed by kaie@kuix.de: https://hg.mozilla.org/comm-central/rev/5cb9862e2908 Backed out changeset efdfb2936679

Fails to build locally for me.

0:08.74 Updating git repository https://github.com/thunderbird/ews-rs.git
0:09.30 error: failed to get ews as a dependency of package ews_xpcom v0.1.0 (/home/user/moz/commcent/mozilla/comm/rust/ews_xpcom)
0:09.30 Caused by:
0:09.30 failed to load source for dependency ews
0:09.30 Caused by:
0:09.31 Unable to update https://github.com/thunderbird/ews-rs.git?rev=ad3c9077987cbf28e9c60a868e5a258e20d4ba70
0:09.31 Caused by:
0:09.31 failed to fetch into: /home/user/.cargo/git/db/ews-rs-fe995326286edf48
0:09.32 Caused by:
0:09.32 attempting to update a git repository, but --frozen was specified

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Sorry about that, I forgot to tb-rust vendor. Should be good to go now.

Target Milestone: --- → 146 Branch

Pushed by kaie@kuix.de:
https://hg.mozilla.org/comm-central/rev/c0fe82ef215b
Allow multiple reply-to addresses on emails. r=babolivier

Status: REOPENED → RESOLVED
Closed: 5 months ago5 months ago
Resolution: --- → FIXED

Comment on attachment 9519383 [details]
Bug 1993555 - Allow multiple reply-to addresses on emails. r=#thunderbird-back-end-reviewers

Uplift Approval Request

  • Please state case for uplift consideration and ensure bug severity is set: This is a fix for an issue that blocks Thunderbird from downloading emails.
  • User impact if declined: This bug causes user emails to fail to download.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Daily?: Yes
  • Has the fix been verified in Beta?: No
  • Needs manual test from QA?: Yes
  • If yes, steps to reproduce: 1. Set up an EWS account.
  1. Send an email to the EWS account with multiple addresses in the reply-to header.
  2. Ensure the email displays correctly when received in Thunderbird.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This change reads multiple items from a header instead of a single header and sets them as a string property on the email in Thunderbird.
  • Does the fix cause any migrations to be skipped?: No
  • String changes made/needed:
Attachment #9519383 - Flags: approval-comm-beta?

Comment on attachment 9519383 [details]
Bug 1993555 - Allow multiple reply-to addresses on emails. r=#thunderbird-back-end-reviewers

[Triage Comment]
Approved for beta

Attachment #9519383 - Flags: approval-comm-beta? → approval-comm-beta+
See Also: → 1996474
Duplicate of this bug: 1996474
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: