Mail-Prioritäts-Header auf Liste gesetzt

Andro · 13. Januar 2021 um 06:01

Ich versende eingehende E-Mails manuell von Gmail. Discourse lehnt sie jedoch mit der Beschwerde ab, dass der Header „Precedence“ auf „list“ gesetzt ist und die E-Mail automatisch generiert wurde – was nicht stimmt: Ich habe sie selbst geschrieben.

Aus den Kopfzeilen:

Auto-Submitted: auto-generated
Precedence: list
List-ID: The Jackrail | Test <test.jackrail.space>
List-Archive: https://jackrail.space/t/a-test-please-work/579
List-Post: mailto:replies+591d4708be667d02b1c5826953683fba@jackrail.space

Was ist hier los?

steinjer · 14. Februar 2021 um 18:48

Seit dem aktuellen Release erfordern E-Mail-Antworten immer eine Antwort an ‘Reply to list’. Eine einfache Antwort durch Klicken auf ‘Reply’ im E-Mail-Programm funktioniert nicht mehr.

Neu in Mailheader ist diese Zeile:
List-Post: <mailto:replies+0d953a3a06191ea09a38f1f4f62129c4@forum.example.org>

Da dies im Code hinzugefügt wurde (geänderte if-Bedingung), funktioniert unser Discourse als Mailinglisten-Server, obwohl wir diese Option nicht ausgewählt haben.

github.com/discourse/discourse

FIX: Group SMTP email improvements (#11633)

master ← issue/group-smtp-email-improvements

merged 05:32AM - 05 Jan 21 UTC

martin-brennan

+75 -12

Fixes a rare race condition causing the `Imap::Sync` class to create an incoming… email and associated post/topic, which then kicks off the PostAlerter to notify others in the PM about a reply in the topic, but for the OP which is not necessary (because the person emailing the IMAP inbox already knows about the OP). Basically, we should never be sending the group SMTP email for the first post in a topic. Also in this PR: * Custom attribute accessors for the to/from/cc addresses on `IncomingEmail`, to parse them from an array to a joined string so the logic for this is only in one place. * Store extra detail against the `IncomingEmail` created in `GroupSmtpMailer` * regex test Mail header Reply-To as string instead of Field, which fixes `warning: deprecated Object#=~ is called on Mail::Field; it always returns nil` * Add DEBUG_IMAP to log all IMAP logs as warnings for easier debugging * Changed the Rails logging to `ImapSyncLog` in the `GroupSmtpMailer`

martin · 14. Februar 2021 um 22:57

Meinst du, dass du ‘Allen antworten’ musst? Ich bin mir nicht sicher, was du mit ‘Antwort an die Liste’ meinst. Welcher Code in diesem PR hätte dieses Verhalten geändert? Die einzige if-Bedingung, die sich geändert hat und die ich sehen kann, ist diese:

github.com/discourse/discourse

lib/email/sender.rb

8da9a3f82


      
          if reply_key.present? && @message.header['Reply-To'].to_s =~ /\<([^\>]+)\>/
            email = Regexp.last_match[1]
            @message.header['List-Post'] = "<mailto:#{email}>"
          end

Aber ich kann nicht erkennen, wie sich das Verhalten nur durch das Hinzufügen von to_s geändert haben sollte?

steinjer · 15. Februar 2021 um 07:58

Ich denke, die geänderte Bedingung der if-Anweisung führt nun zu einer neuen Nachrichtenüberschrift ‘list-post’.

Beispiel für eine neue E-Mail von unserem Discourse, angezeigt in Thunderbird:

Und die E-Mail-Header-Zeile lautet:
List-Post: mailto:replies+6e23071765f50da1c819aba7f2ffb386@forum.sol....

Vor dem neuen Release (keine Antwortliste wird angezeigt):

Und der E-Mail-Header enthält keine Zeile mit „List-Post“.

Thema		Antworten	Aufrufe
Confusion between Reply To and Reply List in email responses UX	45	6195	13. Oktober 2021
Reply-key variable doesn't get expanded in email custom headers Bug	5	927	25. Oktober 2021
Feedback from a community about Mailing List feature General	5	3323	20. April 2015
Allow reply-to individual instead of topic/forum (mailing list feature) Support	56	12638	27. August 2022
Need help with my mailing list Support email , mailing-list	3	391	9. Oktober 2023

Mail-Prioritäts-Header auf Liste gesetzt

Verwandte Themen