Etapas de sincronización IMAP nuevo usuario si Group.incoming_email no está configurado en Group.email_username

thoka · 6 Febrero, 2021 18:18

Mientras jugaba con la sincronización IMAP, se creó un usuario temporal para Group.email_username durante la importación desde la cuenta IMAP.

Al intentar entender qué sucedió, noté que Group.find_by_email no tiene en cuenta Group.email_username, lo que obliga a configurar Group.incoming_email para que incluya al menos este valor.

Pasos para reproducir:

Habilitar la sincronización IMAP
Configurar las opciones de sincronización de correo electrónico en un grupo, pero dejar Group.incoming_email sin configurar
Enviar un correo a Group.email_username

codinghorror · 7 Febrero, 2021 20:08

Creo que IMAP aún está un poco en desarrollo, ¿verdad, @sam?

sam · 8 Febrero, 2021 01:00

Debería tener un soporte razonable; lo utilizamos en producción en Meta.

@martin, ¿puedes echar un vistazo rápido a este informe?

martin · 8 Febrero, 2021 03:44

Gracias @thoka, he revisado esto. Creo que tienes razón en que Group.find_by_email necesita incluir email_username, pero no es la causa de que el usuario de correo del grupo se cree como provisional en este caso. Esto ocurre más adelante en el receptor, cuando añadimos otras direcciones al tema recién creado. Lo importante aquí es 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

Lo cual, a su vez, compara el correo electrónico 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

Creo que necesitaremos modificar esta expresión regular, o crear otra, para que también compare contra email_username. Hasta ahora no hemos considerado un problema que se cree el usuario provisional para el grupo, aunque entiendo que parece un poco desordenado/innecesario. Tendré que realizar algunas pruebas al hacer este cambio para asegurarme de que no se esté creando el usuario provisional para el grupo por alguna razón importante que aún desconozco.

martin · 11 Junio, 2021 01:30

Esto se solucionó en FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub, y la interfaz de configuración de correo electrónico de grupos ha recibido una actualización reciente. Sin embargo, IMAP sigue estando en una etapa muy alfa, así que usa esa función bajo tu propio riesgo.

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

Etapas de sincronización IMAP nuevo usuario si Group.incoming_email no está configurado en Group.email_username

Temas relacionados