Ticket #369 (closed defect: fixed)

Opened 2 years ago

Last modified 17 months ago

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

Changed 2 years ago by dmorton

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

Changed 2 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.

Changed 2 years ago by dmorton

  • status changed from assigned to closed
  • resolution set to fixed

fixed in [1131]

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

Changed 18 months 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...

Changed 17 months ago by buzzz@…

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

Changed 17 months ago by dmorton

see #386

Note: See TracTickets for help on using tickets.