E-Mail-Benachrichtigungen schlagen fehl, wenn doppelte Header vorhanden sind

simonk · 12. April 2022 um 10:37

(Haftungsausschluss: kein Ruby-Programmierer)

Ich halte das für ein besonders unangenehmes Verhalten in der Mail-Bibliothek, die Discourse verwendet. Hier ist die Funktion header_value:

github.com/discourse/discourse

lib/email/sender.rb

9c33f6de0


      
                # a bunch of characters not render correctly in the email
                part content_type: "text/html; charset=utf-8", body: html_part.body.decoded
                part content_type: "text/plain; charset=utf-8", body: text_part.body.decoded
              end
          
              @message.parts.unshift(content)
            end
          end
          
          def header_value(name)
            header = @message.header[name]
            return nil unless header
            header.value
          end
          
          def skip(reason_type, custom_reason: nil)
            attributes = {
              email_type: @email_type,
              to_address: to_address,
              user_id: @user&.id,
              reason_type: reason_type

Soweit ich das beurteilen kann, ruft @message.header[name] diese Methode auf:

https://www.rubydoc.info/github/mikel/mail/Mail%2FHeader:[]

Gemäß RFC können viele Felder mehrfach vorkommen. Wir geben einen String des Werts zurück, wenn nur ein Header vorhanden ist, oder wenn mehr als ein übereinstimmender Header vorhanden ist, geben wir ein Array von Werten zurück, in der Reihenfolge, in der sie im Header von oben nach unten erscheinen.

Discourse setzt automatisch einen Precedence-Header. Da Sie über die Einstellung email_custom_headers einen weiteren hinzufügen, gibt es nun zwei Precedence-Header, und @message.header[\"Precedence\"] gibt ein Array anstelle eines Strings zurück.

Ich glaube, dieser Fehler wird immer dann ausgelöst, wenn email_custom_headers einen Header enthält, der bereits im Nachrichtenobjekt vorhanden ist.

Thema		Antworten	Aufrufe
Notifications are coming again even after reading them [Private Topics plugin] Support notifications	20	1328	1. Dezember 2023
Severe Email Issue since last update a couple days ago - 3.4.0.beta4-dev Support	25	358	16. Februar 2025
Email notifications not sent Support	11	2399	18. November 2016
Discourse sending multiple digest mails to users Support activity-summary	13	2216	15. Januar 2017
Lots of retries mail in sidekiq after upgrade discourse to 3.5.0.beta8-dev( 4893bda14a ) Support	1	35	9. Juli 2025

E-Mail-Benachrichtigungen schlagen fehl, wenn doppelte Header vorhanden sind

Verwandte Themen