Email notifications fail if duplicate headers exist

simonk · April 12, 2022, 10:37am

(Disclaimer: not a ruby programmer)

I think this is particularly nasty behaviour in the mail library that discourse is using. Here’s the header_value function:

github.com

discourse/discourse/blob/9c33f6de05703a42465139fda54773e5fa26c617/lib/email/sender.rb#L434

      
        
                  # 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

As far as I can tell, @message.header[name] is calling this method:

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

As per RFC, many fields can appear more than once, we will return a string of the value if there is only one header, or if there is more than one matching header, will return an array of values in order that they appear in the header ordered from top to bottom.

Discourse automatically sets a Precedence header, so because you’re adding one as well via the email_custom_headers setting, there are now two Precedence headers, and @message.header["Precedence"] is returning an array instead of a string.

I think this bug will be triggered any time email_custom_headers contains a header that already exists on the message object.

Topic		Replies	Views
Notifications are coming again even after reading them [Private Topics plugin] support notifications	20	879	December 1, 2023
Email Hostname Certificate Mismatch Causing sidekiq Queue Overload, Severe Site Instability installation	64	4037	September 30, 2023
Emails have stopped sending - end of file reached support	27	3633	February 23, 2021
Building bootstrap error installation	46	1398	December 20, 2022
Retry error in sidekiq bug	19	1459	June 8, 2022

Email notifications fail if duplicate headers exist

Related Topics