Bug de saut du résumé du chat

merefield · Mars 7, 2025, 2:05

Les e-mails de résumé de chat sont ignorés pour les utilisateurs ayant une adresse e-mail valide :

J’ai effectué une analyse et je pense savoir pourquoi :

Voici finalement où le saut se produit :

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

C’est parce que message.to est vide.

message.to est vide car les arguments envoyés à send_user_email n’incluent pas to_address et celui-ci n’est jamais enrichi avec l’e-mail principal de l’utilisateur, par exemple :

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

Ceci est appelé par Chat Mailer, et vous remarquerez que to_address est manquant :

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

Et c’est peut-être bien, mais au moment où vous arrivez en haut de send_user_email, je pense que to_address devrait être enrichi avec l’e-mail de l’utilisateur afin que lorsqu’il est transmis, l’expéditeur d’e-mails ait accès à cette propriété et que l’envoi d’e-mail réussisse.

Lorsque j’ai ajouté ce petit monkey-patch à send_user_email, le problème disparaît :

      # BUG CORE : si nous ne définissons pas to_address, l'e-mail ne sera finalement pas envoyé et sera ignoré.
      # C'est un bug de base et nous devrons le signaler sur Meta.
      if args[:to_address].blank? && user&.primary_email&.email
        args[:to_address] = user&.primary_email&.email
      end

sam · Mars 9, 2025, 11:07

Merci beaucoup pour le débogage @merefield, je m’assurerai que le membre xp examine cela bientôt.

davidb · Mai 26, 2025, 3:23

J’ai un peu creusé la question, et si nous suivons la piste, il devrait y avoir un enrichissement :

Cela commence par l’emailer ici puis il appelle l’extension de notification utilisateur qui enrichit avec l’email utilisateur ici puis le transmet à l’aide de construction d’email ici et enfin définit l’adresse dans le constructeur de message ici avant de l’envoyer.

En dehors de cela, l’email de résumé de chat fonctionne déjà sur d’autres sites, il y a donc probablement autre chose qui cause un problème ici.

Je pense qu’une solution intermédiaire consiste à transmettre l’email de l’utilisateur au job, ce qui devrait nous donner un peu plus de confiance/visibilité sur le fait que les bonnes données sont présentes dès le départ :

j.jaffeux · Juin 29, 2025, 9:47

Fermer car nous n’avons pas eu de retour sur la correction.

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

Bug de saut du résumé du chat

Sujets connexes