Opened 14 years ago

Closed 14 years ago

Last modified 14 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


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 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 script should tighten its user query to exclude domain-default and system-default accounts.

Change History (6)

comment:1 Changed 14 years ago by dmorton

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

comment:2 Changed 14 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 14 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 14 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 14 years ago by buzzz@…

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

comment:4 Changed 14 years ago by dmorton

see #386

Note: See TracTickets for help on using tickets.