Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#369 closed defect (fixed)

Don't send digests to domain-default or system-default accounts

Reported by: rjl Owned by: dmorton
Priority: normal Milestone: 1.0.2
Component: General Version: 1.0.1
Severity: normal Keywords: digest send-quarantine-digest
Cc:

Description

Currently the settings.php script allows an administrator to set a digest interval for domain-default accounts (e.g. "@domain"), but of course these accounts have no associated email address to send the digest to. The send-quarantine-digests.pl script does not test for this condition either, resulting in blank email addresses when the taint test fails on a "@domain" address. This leads to SMTP errors when the script tries to send the digest with an empty "RCPT TO:".

The settings.php script should not display the digest interval option for domain-default or system-default accounts, only for regular accounts (with "user@domain" addresses). Further, the send-quarantine-digests.pl script should tighten its user query to exclude domain-default and system-default accounts.

Change History (6)

comment:1 Changed 13 years ago by dmorton

But we need a way to specify where the messages *should* go.

comment:2 Changed 12 years ago by dmorton

  • Owner changed from rjl to dmorton
  • Status changed from new to assigned

The digests should probably be sent to the domain admins... but that would

be an enhancement ticket.

comment:3 Changed 12 years ago by dmorton

  • Resolution set to fixed
  • Status changed from assigned to closed

fixed in [1131]

disabling emails to domain defaults for now, and labeled the entry as a default. Translators - please send me patches!

comment:2 Changed 12 years ago by dmorton

In addition, this setting should be used to inherit default settings for newly created users and so maybe should be marked as such...

comment:3 Changed 12 years ago by buzzz@…

Would not be better to let the domain digest mails to be sended to $admin_email ?

comment:4 Changed 12 years ago by dmorton

see #386

Note: See TracTickets for help on using tickets.