Estágios de sincronização IMAP novo usuário se Group.incoming_email não estiver definido como Group.email_username

thoka · Fevereiro 6, 2021, 6:18pm

Ao testar a sincronização IMAP, um usuário temporário foi criado para Group.email_username durante a importação da conta IMAP.

Ao tentar entender o que aconteceu, notei que Group.find_by_email não considera Group.email_username, exigindo que Group.incoming_email seja definido para incluir esse valor, pelo menos.

Passos para reproduzir:

ative a sincronização IMAP
configure as opções de sincronização de e-mail em um grupo, mas deixe Group.incoming_email sem configuração
envie um e-mail para Group.email_username

codinghorror · Fevereiro 7, 2021, 8:08pm

Acredito que o IMAP ainda esteja um pouco em desenvolvimento, @sam?

sam · Fevereiro 8, 2021, 1:00am

Deveria ser razoavelmente suportado, pois o utilizamos em produção no Meta.

@martin, você pode dar uma rápida olhada neste relatório?

martin · Fevereiro 8, 2021, 3:44am

Obrigado @thoka, analisei isso. Acredito que você esteja certo ao dizer que Group.find_by_email precisa ter o email_username incluído, mas esse não é o motivo pelo qual o usuário de email do grupo está sendo criado como provisório neste caso. Isso na verdade ocorre mais adiante no receiver, quando adicionamos outros endereços ao tópico recém-criado. A parte importante aqui é 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

O que, por sua vez, compara o email contra 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

Acho que precisaremos ajustar essa regex, ou outra, para que também compare o email_username. Até agora, não consideramos um problema que o usuário provisório do grupo seja criado, embora eu veja que isso pareça um pouco confuso/desnecessário. Vou precisar fazer alguns testes ao alterar isso para garantir que o usuário provisório do grupo não esteja sendo criado por algum motivo importante que eu ainda desconheço.

martin · Junho 11, 2021, 1:30am

Isso foi corrigido em FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub, e a interface de configuração de e-mail do grupo passou por uma reformulação recente. No entanto, o IMAP ainda está em estado alfa, então use essa funcionalidade por sua conta e risco.

Tópico		Respostas	Visualizações
Email into group sends empty notification Bug	12	2003	20 de Janeiro de 2016
IMAP support for group inboxes Announcements new-feature	32	32087	12 de Janeiro de 2026
When an existing staged user Joins my site, filled in user custom fields data isn't saved Bug	13	1070	28 de Junho de 2023
How to disable mail for staged users? Support	5	821	14 de Novembro de 2024
Allow incoming email for automatic groups Feature groups , email-in	2	30	31 de Março de 2026

Estágios de sincronização IMAP novo usuário se Group.incoming_email não estiver definido como Group.email_username

Tópicos relacionados