Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Traffic Cronjob sends mail violating DMARC rules (similar issue to #1250) #1289

Open
alexlehm opened this issue Oct 24, 2024 · 2 comments
Open

Comments

@alexlehm
Copy link

alexlehm commented Oct 24, 2024

Describe the bug
lib/Froxlor/Cron/Traffic/ReportsCron.php uses the admin address as From address which makes the mail undeliverable for strict DMARC domains, the mail is with subject "Sie erreichen bald Ihr Traffic-Limit"

E.g. an admin uses [email protected] as admin address and the recipient address of the site owner is [email protected], that means that the mail will be rejected by Outlook since it is not DKIM signed and the source domain is set to p=reject

System information

  • Froxlor version: 2.2.4
  • Web server: apache2
  • DNS server: Bind
  • POP/IMAP server: Dovecot
  • SMTP server: postfix
  • FTP server: proftpd
  • OS/Version: Debian 11

To Reproduce
Steps to reproduce the behavior:

  1. Create a reseller with an email address of a strict DMARC domain (p=reject) (that is not hosted on the Froxlor host)
  2. Create at site owner with an email address on outlook.com (or another domain that enforces DMARC rules)
  3. wait until 1st of the next month
  4. the bounce email should go to the postmaster of the site running Froxlor with the rejection reason "Access denied, sending domain [SOMEDOMAIN.DE] does not pass DMARC verification and has a DMARC policy of reject."

Expected behavior
The mail should use a local email address as From, e.g. [email protected]

Additional context
A similar issue was fixed as #1250

@d00p
Copy link
Member

d00p commented Oct 24, 2024

Yes that's correct, there is no SMTP setting yet per froxlor-admin for their adress to send mail.

We can either change that behavior to use the system-sender (if set up with smtp) - without smtp credentials, this problem cannot be fixed

@alexlehm
Copy link
Author

alexlehm commented Oct 24, 2024

I guess it could just be the site-wide from email and use the name of the admin as real-name field
Or maybe have it switchable to use either the owner domain address or the global admin address of Froxlor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants