Error al omitir resumen del chat

merefield · 7 Marzo, 2025 14:05

Los correos electrónicos de resumen de chat se omiten para los usuarios con un correo electrónico válido:

He realizado un análisis y creo que sé por qué:

Aquí es donde finalmente ocurre el salto:

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

Esto se debe a que message.to está en blanco.

message.to está en blanco porque los argumentos enviados a send_user_email no incluyen to_address y esto nunca se enriquece con el correo electrónico principal del usuario, por ejemplo:

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

Esto es llamado por Chat Mailer, y notará que falta 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

Y eso está bien, pero para cuando llegas a la parte superior de send_user_email, creo que to_address debería enriquecerse con el correo electrónico del usuario para que, cuando se pase, el remitente de correo electrónico tenga acceso a esta propiedad y el envío de correo electrónico tenga éxito.

Cuando agregué este pequeño parche a send_user_email, el problema desaparece:

      # BUG PRINCIPAL: si no establecemos to_address, finalmente el correo electrónico no se enviará y se omitirá.
      # Este es un error principal y tendremos que plantearlo en Meta.
      if args[:to_address].blank? && user&.primary_email&.email
        args[:to_address] = user&.primary_email&.email
      end

sam · 9 Marzo, 2025 23:07

Muchas gracias por la depuración @merefield, me aseguraré de que el miembro de xp lo revise pronto.

davidb · 26 Mayo, 2025 15:23

Investigué un poco y, si seguimos el rastro, debería haber un enriquecimiento:

Comienza desde el emisor de correo aquí y luego llama a la extensión de notificación de usuario que enriquece con el correo electrónico del usuario aquí y luego lo pasa al ayudante de construcción de correo aquí y finalmente establece la dirección en el constructor de mensajes aquí antes de enviarlo.

Aparte de eso, el correo electrónico de resumen del chat ya funciona en otros sitios, por lo que es probable que algo más esté causando un problema aquí.

Creo que una solución provisional es pasar el correo electrónico del usuario al trabajo, lo que debería darnos un poco más de confianza/visibilidad de que los datos correctos están allí desde el principio:

j.jaffeux · 29 Junio, 2025 21:47

Cerrar porque no hemos recibido comentarios sobre la corrección.

Tema		Respuestas	Vistas
Notification emails not being sent for Chat as per user preference Bug chat	27	1480	4 Julio 2023
Prevent Emailed Chat Notifications when invoked via SDK Feature chat , completed	5	250	15 Julio 2024
Digest/Activity Summary emails not sending for users even though all conditions are met (Discourse 3.6) Support email , activity-summary	16	162	3 Noviembre 2025
[Paid] Digest emails are not being sent to all users as they should Marketplace activity-summary	3	744	15 Abril 2020
Activity Summary not sent if other emails are sent Feature email , completed , activity-summary	19	913	18 Julio 2024

Error al omitir resumen del chat

Temas relacionados