IMAP-Synchronisierungsphasen neuer Benutzer, wenn Group.incoming_email nicht auf Group.email_username gesetzt ist

thoka · 6. Februar 2021 um 18:18

Während des Spielerischen Ausprobierens der IMAP-Synchronisation wurde während des Imports aus dem IMAP-Konto ein gestaffelter Benutzer für Group.email_username erstellt.

Beim Versuch zu verstehen, was passiert ist, habe ich festgestellt, dass Group.find_by_email Group.email_username nicht berücksichtigt, was erfordert, dass Group.incoming_email mindestens diesen Wert enthält.

Reproduktionsschritte:

IMAP-Synchronisation aktivieren
E-Mail-Synchronisierungsoptionen für eine Gruppe konfigurieren, aber Group.incoming_email unkonfiguriert lassen
Eine E-Mail an Group.email_username senden

codinghorror · 7. Februar 2021 um 20:08

Ich glaube, IMAP ist bei @sam immer noch etwas in Arbeit?

sam · 8. Februar 2021 um 01:00

Es sollte gut unterstützt sein, da wir es in der Produktion bei Meta einsetzen.

@martin, könntest du diesen Bericht kurz prüfen?

martin · 8. Februar 2021 um 03:44

Danke @thoka, ich habe mir das angesehen. Ich denke, du liegst richtig, dass Group.find_by_email das email_username enthalten muss, aber das ist in diesem Fall nicht die Ursache dafür, dass der Gruppen-E-Mail-Benutzer gestaged wird. Dies geschieht tatsächlich später im Receiver, wo wir weitere Adressen zum neu erstellten Thema hinzufügen. Der wichtige Teil hier ist should_invite?

github.com/discourse/discourse

lib/email/receiver.rb

e9364856f


      
          def add_other_addresses(post, sender)
            %i(to cc bcc).each do |d|
              if @mail[d] && @mail[d].address_list && @mail[d].address_list.addresses
                @mail[d].address_list.addresses.each do |address_field|
                  begin
                    address_field.decoded
                    email = address_field.address.downcase
                    display_name = address_field.display_name.try(:to_s)
                    next unless email["@"]
                    if should_invite?(email)
                      user = find_or_create_user(email, display_name)
                      if user && can_invite?(post.topic, user)
                        post.topic.topic_allowed_users.create!(user_id: user.id)
                        TopicUser.auto_notification_for_staging(user.id, post.topic_id, TopicUser.notification_reasons[:auto_watch])
                        post.topic.add_small_action(sender, "invited_user", user.username, import_mode: @opts[:import_mode])
                      end
                      # cap number of staged users created per email
                      if @staged_users.count > SiteSetting.maximum_staged_users_per_email
                        post.topic.add_moderator_post(sender, I18n.t("emails.incoming.maximum_staged_user_per_email_reached"), import_mode: @opts[:import_mode])
                        return

This file has been truncated. show original

github.com/discourse/discourse

lib/email/receiver.rb

e9364856f


      
          def should_invite?(email)
            email !~ Email::Receiver.reply_by_email_address_regex &&
            email !~ group_incoming_emails_regex &&
            email !~ category_email_in_regex
          end

Dies vergleicht wiederum die E-Mail-Adresse mit group_incoming_emails_regex:

github.com/discourse/discourse

lib/email/receiver.rb

e9364856f


      
          def group_incoming_emails_regex
            @group_incoming_emails_regex ||= Regexp.union Group.pluck(:incoming_email).select(&:present?).map { |e| e.split("|") }.flatten.uniq
          end

Ich denke, wir müssen sicherstellen, dass wir in diesem regulären Ausdruck oder einem anderen auch das email_username vergleichen. Bisher haben wir es nicht als Problem empfunden, dass der gestagte Benutzer für die Gruppe erstellt wird, obwohl ich verstehe, dass dies etwas unübersichtlich/unnötig aussieht. Ich muss einige Tests durchführen, wenn ich dies ändere, um sicherzustellen, dass der gestagte Benutzer für die Gruppe nicht aus einem wichtigen, mir bisher unbekannten Grund erstellt wird.

martin · 11. Juni 2021 um 01:30

Dies wurde durch FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub behoben, und die Benutzeroberfläche für die Gruppen-E-Mail-Konfiguration wurde kürzlich überarbeitet. IMAP befindet sich jedoch nach wie vor in einem Alpha-Stadium, daher verwenden Sie diese Funktion auf eigene Gefahr.

Thema		Antworten	Aufrufe
Email into group sends empty notification Bug	12	2003	20. Januar 2016
IMAP support for group inboxes Announcements new-feature	32	32087	12. Januar 2026
When an existing staged user Joins my site, filled in user custom fields data isn't saved Bug	13	1070	28. Juni 2023
How to disable mail for staged users? Support	5	821	14. November 2024
Allow incoming email for automatic groups Feature groups , email-in	2	30	31. März 2026

IMAP-Synchronisierungsphasen neuer Benutzer, wenn Group.incoming_email nicht auf Group.email_username gesetzt ist

Verwandte Themen