Ошибка пропуска в сводке чата

merefield · 07.Март.2025 14:05:13

Электронные письма с кратким итогом чата пропускаются для пользователей с действительным адресом электронной почты:

Я провёл анализ и считаю, что знаю причину:

Вот место, где происходит пропуск:

lib/email/sender.rb

ffdc97f37


      
                 begin
                   @message.message
                 rescue StandardError
                   nil
                 end
               )
            return
          end
          
          return skip(SkippedEmailLog.reason_types[:sender_message_blank]) if @message.blank?
          return skip(SkippedEmailLog.reason_types[:sender_message_to_blank]) if @message.to.blank?
          
          if SiteSetting.disable_emails == "non-staff" && !bypass_disable
            return unless find_user&.staff?
          end
          
          if to_address.end_with?(".invalid")
            return skip(SkippedEmailLog.reason_types[:sender_message_to_invalid])
          end
          
          if @message.text_part

Это происходит потому, что message.to пуст.

message.to пуст, потому что аргументы, переданные в send_user_email, не включают to_address, и это значение никогда не дополняется основным адресом электронной почты пользователя, например:

args: {"type"=>"chat_summary", "user_id"=>3, "force_respect_seen_recently"=>true, "current_site_id"=>"default"}

github.com/discourse/discourse

app/jobs/regular/user_email.rb

ffdc97f37


      
            args[:type] = args[:type].to_s
          
            send_user_email(args)
          
            if args[:type] == "digest"
              # Record every attempt at sending a digest email, even if it was skipped
              UserStat.where(user_id: args[:user_id]).update_all(digest_attempted_at: Time.current)
            end
          end
          
          def send_user_email(args)
            post = nil
            notification = nil
            type = args[:type]
            user = User.find_by(id: args[:user_id])
            to_address =
              args[:to_address].presence || user&.primary_email&.email.presence || "no_email_found"
          
            set_skip_context(type, args[:user_id], to_address, args[:post_id])
          
            return skip(SkippedEmailLog.reason_types[:user_email_no_user]) if !user

Это вызывается Chat Mailer, и вы заметите, что to_address отсутствует:

github.com/discourse/discourse

plugins/chat/lib/chat/mailer.rb

ffdc97f37


      
            return unless SiteSetting.chat_enabled
          
            User
              .real
              .activated
              .not_staged
              .not_suspended
              .where(id: users_with_unreads)
              .find_each do |user|
                if DiscoursePluginRegistry.apply_modifier(:chat_mailer_send_summary_to_user, true, user)
                  Jobs.enqueue(
                    :user_email,
                    type: "chat_summary",
                    user_id: user.id,
                    force_respect_seen_recently: true,
                  )
                end
              end
          end
          
          private

И это, возможно, допустимо, но к моменту входа в send_user_email, по моему мнению, to_address должно быть дополнено адресом электронной почты пользователя, чтобы при передаче дальше почтовый агент имел доступ к этому свойству, и отправка письма успешно выполнялась.

Когда я добавил этот небольшой патч (monkey-patch) в send_user_email, проблема исчезла:

      # CORE BUG: если мы не установим to_address, в итоге письмо не будет отправлено и будет пропущено.
      # Это критическая ошибка, и нам нужно сообщить о ней на Meta.
      if args[:to_address].blank? && user&.primary_email&.email
        args[:to_address] = user&.primary_email&.email
      end

sam · 09.Март.2025 23:07:56

Огромное спасибо за отладку, @merefield. Убедимся, что члены XP скоро посмотрят на это.

davidb · 26.Май.2025 15:23:17

Я немного разобрался в этом, и если проследить цепочку, должно происходить обогащение данных:

Всё начинается с email-сервиса здесь, затем он вызывает расширение уведомлений пользователя, которое обогащает данные email-адресом пользователя здесь, после чего передаёт их в вспомогательный метод построения email здесь, и, наконец, устанавливает адрес в конструкторе сообщения здесь перед отправкой.

Помимо этого, сводный email чата уже работает на других сайтах, поэтому, скорее всего, проблема вызвана чем-то другим.

Я считаю, что временным решением будет передача email-адреса пользователя в задачу, что должно повысить нашу уверенность и прозрачность в том, что правильные данные присутствуют изначально:

j.jaffeux · 29.Июнь.2025 21:47:06

Закрыть, так как мы не получили обратной связи по исправлению

Тема		Ответов	Просм.
Notification emails not being sent for Chat as per user preference Bug chat	27	1578	04.07.2023
Prevent Emailed Chat Notifications when invoked via SDK Feature chat , completed	5	275	15.07.2024
Digest/Activity Summary emails not sending for users even though all conditions are met (Discourse 3.6) Support activity-summary , email	19	305	03.12.2025
[Paid] Digest emails are not being sent to all users as they should Marketplace activity-summary	3	761	15.04.2020
Activity Summary not sent if other emails are sent Feature completed , email , activity-summary	19	961	18.07.2024

Ошибка пропуска в сводке чата

Связанные темы