Nuove fasi di sincronizzazione IMAP per un nuovo utente se Group.incoming_email non è impostato su Group.email_username

thoka · 6 Febbraio 2021, 6:18pm

Mentre provavo la sincronizzazione IMAP, è stato creato un utente in staging per Group.email_username durante l’importazione dall’account IMAP.
Nel tentativo di capire cosa sia successo, ho notato che Group.find_by_email non tiene conto di Group.email_username, rendendo necessario impostare Group.incoming_email in modo che includa almeno questo valore.

Passi per riprodurre il problema:

abilita la sincronizzazione IMAP
configura le opzioni di sincronizzazione email su un gruppo, ma lascia Group.incoming_email non configurato
invia una mail a Group.email_username

codinghorror · 7 Febbraio 2021, 8:08pm

Credo che IMAP sia ancora un po’ in fase di sviluppo, @sam?

sam · 8 Febbraio 2021, 1:00am

Dovrebbe essere supportato in modo ragionevole: lo utilizziamo in produzione su Meta.

@martin, puoi dare un’occhiata veloce a questa segnalazione?

martin · 8 Febbraio 2021, 3:44am

Grazie @thoka, ho dato un’occhiata a questo. Penso che tu abbia ragione nel dire che Group.find_by_email debba includere email_username, ma non è la causa per cui l’utente email del gruppo viene creato in staging in questo caso. Questo accade in realtà più avanti nel ricevitore, dove aggiungiamo altri indirizzi al topic appena creato. La parte importante qui è 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

Che a sua volta confronta l’email con 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

Penso che dovremo modificare questa espressione regolare, o un’altra, in modo da confrontare anche email_username. Finora non abbiamo riscontrato problemi nel fatto che venga creato l’utente in staging per il gruppo, anche se capisco che possa sembrare un po’ disordinato o inutile. Dovrò effettuare alcuni test quando apporterò questa modifica per assicurarmi che l’utente in staging per il gruppo non venga creato per qualche motivo importante attualmente sconosciuto (a me).

martin · 11 Giugno 2021, 1:30am

Questo è stato risolto da FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub e l’interfaccia utente per la configurazione delle email dei gruppi è stata recentemente rinnovata. Tuttavia, IMAP è ancora in una fase molto alpha, quindi utilizzalo a tuo rischio e pericolo.

Argomento		Risposte	Visualizzazioni
Email into group sends empty notification Bug	12	2003	Gennaio 20, 2016
IMAP support for group inboxes Announcements new-feature	32	32087	Gennaio 12, 2026
When an existing staged user Joins my site, filled in user custom fields data isn't saved Bug	13	1070	Giugno 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	Marzo 31, 2026

Nuove fasi di sincronizzazione IMAP per un nuovo utente se Group.incoming_email non è impostato su Group.email_username

Argomenti correlati