Chat-Zusammenfassung Überspringen Fehler

merefield · 7. März 2025 um 14:05

Chat-Zusammenfassungs-E-Mails werden für Benutzer mit einer gültigen E-Mail übersprungen:

Ich habe einige Analysen durchgeführt und glaube, den Grund zu kennen:

Hier ist letztendlich, wo das Überspringen passiert:

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

Dies liegt daran, dass message.to leer ist.

message.to ist leer, da die an send_user_email gesendeten Argumente keine to_address enthalten und dies nie mit der primären E-Mail des Benutzers angereichert wird, z. B.:

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

Dies wird vom Chat Mailer aufgerufen, und Sie werden feststellen, dass to_address fehlt:

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

und das ist vielleicht in Ordnung, aber bis Sie oben in send_user_email ankommen, glaube ich, dass to_address mit der E-Mail des Benutzers angereichert werden sollte, damit der E-Mail-Versender beim Weiterleiten Zugriff auf diese Eigenschaft erhält und der E-Mail-Versand erfolgreich ist.

Als ich diesen kleinen Monkey-Patch zu send_user_email hinzufügte, verschwand das Problem:

      # CORE BUG: if we don't set to_address, ultimately the email won't send and will be skipped.
      # This is a core bug and we will need to raise it on Meta.
      if args[:to_address].blank? && user&.primary_email&.email
        args[:to_address] = user&.primary_email&.email
      end

sam · 9. März 2025 um 23:07

Vielen Dank für das Debugging @merefield, wir werden sicherstellen, dass sich das Mitglied xp bald darum kümmert.

davidb · 26. Mai 2025 um 15:23

Ich habe mich ein wenig damit beschäftigt, und wenn wir der Spur folgen, sollte es eine Anreicherung geben:

Es beginnt mit dem E-Mail-Versender hier und dann ruft es die Benutzerbenachrichtigungserweiterung hier auf, die mit der Benutzer-E-Mail hier angereichert wird und sie dann an den E-Mail-Hilfshelfer hier weitergibt und schließlich die Adresse im Nachrichten-Builder hier festlegt, bevor er sie sendet.

Abgesehen davon funktioniert die Chat-Zusammenfassungs-E-Mail bereits auf anderen Websites, daher gibt es hier wahrscheinlich ein anderes Problem.

Ich denke, eine Zwischenlösung besteht darin, die Benutzer-E-Mail an den Job zu übergeben, was uns von Anfang an mehr Vertrauen/Sichtbarkeit gibt, dass die richtigen Daten vorhanden sind:

j.jaffeux · 29. Juni 2025 um 21:47

Schließen, da wir kein Feedback zur Korrektur erhalten haben

Thema		Antworten	Aufrufe
Notification emails not being sent for Chat as per user preference Bug chat	27	1480	4. Juli 2023
Prevent Emailed Chat Notifications when invoked via SDK Feature chat , completed	5	250	15. Juli 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. November 2025
[Paid] Digest emails are not being sent to all users as they should Marketplace activity-summary	3	744	15. April 2020
Activity Summary not sent if other emails are sent Feature email , completed , activity-summary	19	913	18. Juli 2024

Chat-Zusammenfassung Überspringen Fehler

Verwandte Themen