Project

General

Profile

Actions

Defect #743

closed

SMTP error when list of recipients is empty

Added by Marc Dequènes about 16 years ago. Updated about 16 years ago.

Status:
Closed
Priority:
Low
Assignee:
-
Category:
Email notifications
Target version:
-
Start date:
2008-02-26
Due date:
% Done:

100%

Estimated time:
Resolution:
Affected version:

Description

I found on my local MTA:
Feb 26 15:00:57 jones postfix/smtpd13527: warning: Illegal address syntax from localhost[127.0.0.1] in RCPT command: <>
This is due to the user updating an issue but with mail preferences set to not mail himself for its own changes. As there is nobody watching the issue, the recipient list is empty.

In 'app/models/mailer.rb' create_mail(), super is called even if there is nothing to mail, causing this error. Perhaps something like :
super unless recipients.empty? and (not cc or cc.empty?) and (not bcc or bcc.empty?)
would do, but i'm not sure the Bcc-recipients feature would not cause the same SMTP error too (perhaps setting recipients = from would solve the issue in this case).

Actions #1

Updated by Jean-Philippe Lang about 16 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

Applied in changeset r1174.

Actions

Also available in: Atom PDF