#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 14 years ago by dmorton
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
But we need a way to specify where the messages *should* go.