Closed Bug 1738890 Opened 3 years ago Closed 3 years ago

send mail hangs forever while posting news entry with to: header - TypeError: messageFile is null SmtpService.jsm

Categories

(MailNews Core :: Networking: SMTP, defect)

Thunderbird 91
Unspecified
All
defect

Tracking

(thunderbird_esr91+ fixed, thunderbird95+ fixed)

RESOLVED FIXED
96 Branch
Tracking Status
thunderbird_esr91 + fixed
thunderbird95 + fixed

People

(Reporter: ste.sachse, Assigned: rnons)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.69 Safari/537.36

Steps to reproduce:

send news post with to: header

Actual results:

news is posted fine, but the email is not send. send dialog hangs forever. there is a logmessage

TypeError: messageFile is null SmtpService.jsm:121:9 (resource:///modules/SmtpService.jsm)

Expected results:

mail should be sent

Component: Untriaged → Networking: SMTP
Keywords: regression
Product: Thunderbird → MailNews Core

Confirmed

Blocks: tb91found
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(remotenonsense)
OS: Unspecified → All
Summary: send mail hangs forever while posting news entry with to: header → send mail hangs forever while posting news entry with to: header - TypeError: messageFile is null SmtpService.jsm

Here the parameter (file) seems to be missing:
https://searchfox.org/comm-central/rev/37baa8dd18341e3dd3083347c3b91292b6cddcc9/mailnews/compose/src/MessageSend.jsm#624

    this._deliverFileAsMail();

=> this._deliverFileAsMail(this._deliveryFile);

That seems a little too easy to me!?

Assignee: nobody → remotenonsense
Status: NEW → ASSIGNED

Thanks Alfred.

Flags: needinfo?(remotenonsense)
Regressed by: 1211292
Target Milestone: --- → 96 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/d03a0ec5d8a3
Fix sending mail together with posting news. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

Comment on attachment 9249658 [details]
Bug 1738890 - Fix sending mail together with posting news. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #): 1211292
User impact if declined: When sending mail together with posting news, the mail will fail
Testing completed (on c-c, etc.): c-c
Risk to taking this patch (and alternatives if risky): low

Attachment #9249658 - Flags: approval-comm-esr91?
Attachment #9249658 - Flags: approval-comm-beta?

Comment on attachment 9249658 [details]
Bug 1738890 - Fix sending mail together with posting news. r=mkmelin

[Triage Comment]
Approved for beta

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

Comment on attachment 9249658 [details]
Bug 1738890 - Fix sending mail together with posting news. r=mkmelin

[Triage Comment]
Approved for ers91

Attachment #9249658 - Flags: approval-comm-esr78+

Comment on attachment 9249658 [details]
Bug 1738890 - Fix sending mail together with posting news. r=mkmelin

[Triage Comment]
correcting flag

Attachment #9249658 - Flags: approval-comm-esr91?
Attachment #9249658 - Flags: approval-comm-esr91+
Attachment #9249658 - Flags: approval-comm-esr78+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: