Bug de pulo do resumo do chat

merefield · Março 7, 2025, 2:05pm

Os e-mails de Resumo de Chat estão sendo ignorados para usuários com um e-mail válido:

Fiz algumas análises e acredito saber o motivo:

Aqui é onde o pulo acontece:

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

Isso ocorre porque message.to está em branco.

message.to está em branco porque os argumentos enviados para send_user_email não incluem to_address e isso nunca é enriquecido com o e-mail principal do usuário, por exemplo:

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

Isso é chamado pelo Chat Mailer, e você notará que to_address está faltando:

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

E isso talvez esteja tudo bem, mas quando você chega ao topo de send_user_email, acredito que to_address deveria ser enriquecido com o e-mail do usuário para que, quando ele for passado adiante, o remetente receba acesso a essa propriedade e o envio de e-mail seja bem-sucedido.

Quando adicionei este pequeno monkey-patch a send_user_email, o problema desaparece:

      # BUG CORE: se não definirmos to_address, eventualmente o e-mail não será enviado e será ignorado.
      # Este é um bug core e precisaremos levantá-lo no Meta.
      if args[:to_address].blank? && user&.primary_email&.email
        args[:to_address] = user&.primary_email&.email
      end

sam · Março 9, 2025, 11:07pm

Obrigado pelas dicas de depuração, @merefield. Certificarei-me de que a equipe de experiência do membro analise isso em breve.

davidb · Maio 26, 2025, 3:23pm

Eu investiguei um pouco isso e, se seguirmos o rastro, deve haver um enriquecimento:

Começa do emailer aqui e então chama a extensão de notificação do usuário que enriquece com o email do usuário aqui e então o passa para o helper de construção de email aqui e finalmente define o endereço no construtor de mensagens aqui antes de enviá-lo.

Além disso, o email de resumo do chat já funciona em outros sites, então provavelmente há algo mais causando um problema aqui.

Acho que uma solução provisória é passar o email do usuário para o job, o que nos dará um pouco mais de confiança/visibilidade de que os dados corretos estão lá desde o início:

j.jaffeux · Junho 29, 2025, 9:47pm

Fechar, pois não recebemos feedback sobre a correção

Tópico		Respostas	Visualizações
Notification emails not being sent for Chat as per user preference Bug chat	27	1480	4 de Julho de 2023
Prevent Emailed Chat Notifications when invoked via SDK Feature chat , completed	5	250	15 de Julho de 2024
Digest/Activity Summary emails not sending for users even though all conditions are met (Discourse 3.6) Support email , activity-summary	16	163	3 de Novembro de 2025
[Paid] Digest emails are not being sent to all users as they should Marketplace activity-summary	3	744	15 de Abril de 2020
Activity Summary not sent if other emails are sent Feature email , completed , activity-summary	19	913	18 de Julho de 2024

Bug de pulo do resumo do chat

Tópicos relacionados