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

Notification Mails detected as SPAM #1077

Closed
dasGoogle opened this issue Sep 22, 2023 · 4 comments · Fixed by #1078
Closed

Notification Mails detected as SPAM #1077

dasGoogle opened this issue Sep 22, 2023 · 4 comments · Fixed by #1078
Labels
[C] bug Something isn't working

Comments

@dasGoogle
Copy link

dasGoogle commented Sep 22, 2023

Describe the bug
This is likely specific to the exact email setup users have but in my case, the notification mails in my case are caught by the SpamAssassin rule TO_NO_BRKTS_HTML_IMG which is discussed here. As a fix for this, it would be useful if ephios used the configured first and last name to properly address the users in the To: field of the email.

To Reproduce
Make ephios send a notification email.

Expected behavior
Emails should arrive in my inbox without being detected as SPAM. This could be done by modifying the To: field.

Screenshots
-

Environment
An email SPAM filter using SpamAssassin

Additional context
-

@dasGoogle dasGoogle added the [C] bug Something isn't working label Sep 22, 2023
@felixrindt
Copy link
Member

You mentioned more factors in other channels like only having one embedded image. Can you explain that here too?

@dasGoogle
Copy link
Author

Basically, all my knowledge comes from the StackOverflow thread linked in the initial issue, which also goes into a single image and an HTML MIME part being in the mail.

@felixrindt
Copy link
Member

@dasGoogle can you please check #1078 resolved your issue?

@dasGoogle
Copy link
Author

Just deployed the new Docker container and it works perfectly. Receiving HTML and Plain Text mails without any SPAM flags. Thanks for the quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants