Synchronisation IMAP : nouvel utilisateur si Group.incoming_email n'est pas défini sur Group.email_username

thoka · Février 6, 2021, 6:18

Lors de tests de la synchronisation IMAP, un utilisateur temporaire a été créé pour Group.email_username lors de l’importation depuis le compte IMAP.
En essayant de comprendre ce qui s’est passé, j’ai remarqué que Group.find_by_email ne prend pas Group.email_username en compte, ce qui oblige à configurer Group.incoming_email pour inclure au moins cette valeur.

Étapes pour reproduire le problème :

Activer la synchronisation IMAP
Configurer les options de synchronisation des e-mails pour un groupe, mais laisser Group.incoming_email non configuré
Envoyer un e-mail à Group.email_username

codinghorror · Février 7, 2021, 8:08

Je pense que l’IMAP est encore un peu en cours de développement, @sam ?

sam · Février 8, 2021, 1:00

Cela devrait être raisonnablement pris en charge, nous l’utilisons en production chez Meta.

@martin, peux-tu jeter un coup d’œil rapide à ce rapport ?

martin · Février 8, 2021, 3:44

Merci @thoka, j’ai examiné cela. Je pense que vous avez raison : Group.find_by_email doit inclure email_username, mais ce n’est pas la cause de la mise en file d’attente de l’utilisateur du groupe dans ce cas. Cela se produit en réalité plus tard dans le récepteur, lorsque nous ajoutons d’autres adresses au nouveau sujet créé. L’élément clé ici est 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

Ce qui, à son tour, compare l’adresse e-mail à 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

Je pense que nous devrons modifier cette expression régulière, ou en créer une autre, pour comparer également avec email_username. Jusqu’à présent, nous n’avons pas constaté que la création de l’utilisateur mis en file d’attente pour le groupe posait problème, bien que je reconnaisse que cela semble un peu désordonné ou inutile. Je devrai effectuer quelques tests lors de cette modification pour m’assurer que l’utilisateur mis en file d’attente pour le groupe n’est pas créé pour une raison importante actuellement inconnue (de ma part).

martin · Juin 11, 2021, 1:30

Cela a été corrigé par FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub, et l’interface de configuration des courriels de groupe a récemment été entièrement repensée. Cependant, IMAP est toujours dans un état alpha, donc utilisez cette fonctionnalité à vos propres risques.

Sujet		Réponses	Vues
Email into group sends empty notification Bug	12	2003	Janvier 20, 2016
IMAP support for group inboxes Announcements new-feature	32	32087	Janvier 12, 2026
When an existing staged user Joins my site, filled in user custom fields data isn't saved Bug	13	1070	Juin 28, 2023
How to disable mail for staged users? Support	5	821	Novembre 14, 2024
Allow incoming email for automatic groups Feature groups , email-in	2	30	Mars 31, 2026

Synchronisation IMAP : nouvel utilisateur si Group.incoming_email n'est pas défini sur Group.email_username

Sujets connexes