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

Do not send autoreplies when Precedence: bulk #4363

Closed
stsimb opened this issue Nov 22, 2024 · 5 comments
Closed

Do not send autoreplies when Precedence: bulk #4363

stsimb opened this issue Nov 22, 2024 · 5 comments

Comments

@stsimb
Copy link

stsimb commented Nov 22, 2024

Hello!

Today we got in a mail loop because freescout (correctly) sent autoreply emails to emails originating from mailman mailing list requests.

Message-ID: mailman.0.1732265297.10216.colloq@cs.nyu.edu
Date: Fri, 22 Nov 2024 03:48:17 -0500
Precedence: bulk
X-BeenThere: colloq@cs.nyu.edu
X-Mailman-Version: 2.1.15

Is it possible to not send auto-replies when the source has the header "Precedence: bulk" or "X-Mailman-Version" ?

Thanks!

@freescout-help
Copy link
Collaborator

freescout-help commented Nov 22, 2024

"Precedence: bulk" is already there: https://github.com/freescout-help-desk/freescout/blob/dist/app/Misc/Mail.php#L594

Make sure you are using the latest FreeScout version.

@stsimb
Copy link
Author

stsimb commented Nov 22, 2024

Thanks for the reply.
We're using version 1.8.152.
Strange, we definitely had autoreplies going out...

@freescout-help
Copy link
Collaborator

To make it clear: if "Precedence: bulk" header is present in the incoming email there will be no auto-reply sent to such email.

@stsimb
Copy link
Author

stsimb commented Nov 25, 2024

You're right.

Investigating further, I found that some of the emails we received had "Precedence: list", and they were the ones that got the auto-replies.

It looks like mailman 2 does that
https://bazaar.launchpad.net/~mailman-coders/mailman/2.1/view/head:/Mailman/Message.py#L289

Could you please amend the freescout code to include this as well?

Thanks!

@freescout-help
Copy link
Collaborator

Fixed in the master branch and will be published in the next release.

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