Vacation/Out of office emails still processed as new replies

bug
1

Continuing the discussion from Out of office replies to Digests are still being processed as new topics:

This is still happening. See this reply @ FeverBee.

1 Like
2

Sure @zogstrip can have a look. It’s usually a wildly misconfigured mail client.

3 Likes
3

Since that email has no header indicating it was auto-generated, that’s unfortunately the best I can do

https://github.com/discourse/discourse/commit/ca4c939e9a2d6c202ab84841d255f00570f02f45

2 Likes
4

I don’t think it would hurt to add

X-Auto-Response-Suppress: All

to the headers of outgoing mail. Github and zendesk both seem to do that.

1 Like
5

Sure, PRs are welcome :wink:

1 Like
6

Do you take patches?


--- lib/email/message_builder.rb.old    2016-08-01 15:58:26.397393755 -0700
+++ lib/email/message_builder.rb        2016-08-01 16:00:24.295659384 -0700
@@ -139,6 +139,9 @@
       result['X-Discourse-Post-Id']  = @opts[:post_id].to_s  if @opts[:post_id]
       result['X-Discourse-Topic-Id'] = @opts[:topic_id].to_s if @opts[:topic_id]
 
+      # auto-replies just need extra work later to ignore
+      result['X-Auto-Response-Suppress'] = 'All'
+
       if allow_reply_by_email?
         result['X-Discourse-Reply-Key'] = reply_key
         result['Reply-To'] = reply_by_email_address

Alternatively it looks like the code would accept just a config change:

--- config/site_settings.yml.old        2016-08-01 16:02:30.394083505 -0700
+++ config/site_settings.yml    2016-08-01 16:03:00.474661911 -0700
@@ -556,7 +556,7 @@
   disable_digest_emails:
     default: false
     client: true
-  email_custom_headers: 'Auto-Submitted: auto-generated'
+  email_custom_headers: 'Auto-Submitted: auto-generated | X-Auto-Response-Suppress: All'
   email_subject: '[%{site_name}] %{optional_pm}%{optional_cat}%{topic_title}'
   reply_by_email_enabled:
     default: false
3 Likes
7

That looks legit:

8

We’d rather take pull request with tests. Anyway, I pushed the fix. Thanks for suggesting that :thumbsup:

https://github.com/discourse/discourse/commit/b08ab829b80794edb9fdb987d394c84a4ba3417e

4 Likes