重複ヘッダーが存在する場合、メール通知が失敗します

simonk · 2022 年 4 月 12 日午前 10:37

(免責事項: Rubyプログラマーではありません)

Discourseが使用しているメールライブラリの非常に厄介な動作だと思います。以下はheader_value関数です。

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

私の理解では、@message.header[name]はこのメソッドを呼び出しています。

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

RFCによると、多くのフィールドは複数回出現する可能性があります。ヘッダーが1つしかない場合は文字列を返し、一致するヘッダーが複数ある場合は、ヘッダーの上から下への順序で値の配列を返します。

Discourseは「Precedence」ヘッダーを自動的に設定するため、email_custom_headers設定を通じて追加しているため、現在2つの「Precedence」ヘッダーがあり、@message.header["Precedence"]は文字列ではなく配列を返しています。

このバグは、email_custom_headersにメッセージオブジェクトに既に存在するヘッダーが含まれている場合にトリガーされると思います。

トピック		返信	表示
Notifications are coming again even after reading them [Private Topics plugin] Support notifications	20	1328	2023 年 12 月 1 日
Severe Email Issue since last update a couple days ago - 3.4.0.beta4-dev Support	25	358	2025 年 2 月 16 日
Email notifications not sent Support	11	2399	2016 年 11 月 18 日
Discourse sending multiple digest mails to users Support activity-summary	13	2216	2017 年 1 月 15 日
Lots of retries mail in sidekiq after upgrade discourse to 3.5.0.beta8-dev( 4893bda14a ) Support	1	35	2025 年 7 月 9 日

重複ヘッダーが存在する場合、メール通知が失敗します

関連トピック