IMAP синхронизация ставит нового пользователя в очередь, если Group.incoming_email не установлен в Group.email_username

thoka · 06.Февраль.2021 18:18:29

При тестировании синхронизации IMAP во время импорта из учётной записи IMAP был создан промежуточный пользователь для Group.email_username.
Пытаясь понять, что произошло, я заметил, что Group.find_by_email не учитывает Group.email_username, что требует как минимум установки Group.incoming_email с включением этого значения.

Шаги для воспроизведения:

включить синхронизацию IMAP
настроить параметры синхронизации электронной почты для группы, но оставить Group.incoming_email не настроенным
отправить письмо на Group.email_username

codinghorror · 07.Февраль.2021 20:08:41

Полагаю, IMAP всё ещё находится в стадии доработки, @sam?

sam · 08.Февраль.2021 01:00:42

Это должно быть достаточно хорошо поддерживаемым, так как мы используем это в продакшене в Meta.

@martin, можешь быстро взглянуть на этот отчёт?

martin · 08.Февраль.2021 03:44:36

Спасибо @thoka, я посмотрел на это. Я думаю, вы правы в том, что в Group.find_by_email нужно добавить email_username, но в данном случае это не является причиной того, что пользователь группы попадает в стадию ожидания. На самом деле это происходит позже в обработчике входящих писем, где мы добавляем другие адреса к только что созданной теме. Важным моментом здесь является 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

Который, в свою очередь, сравнивает адрес электронной почты с 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

Я думаю, нам нужно будет изменить это так, чтобы в этом регулярном выражении (или в другом) мы также сравнивали с email_username. До сих пор создание временного пользователя для группы не вызывало проблем, хотя я понимаю, что это выглядит немного запутанно/лишним. Мне потребуется провести тестирование после внесения изменений, чтобы убедиться, что временный пользователь для группы не создается по какой-то важной, пока неизвестной (мне) причине.

martin · 11.Июнь.2021 01:30:10

Это было исправлено в FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub, а интерфейс настройки групповой электронной почты недавно был значительно обновлён. Тем не менее, IMAP всё ещё находится в альфа-версии, поэтому используйте эту функцию на свой страх и риск.

Тема		Ответов	Просм.
Email into group sends empty notification Bug	12	2012	20.01.2016
IMAP support for group inboxes Announcements new-feature	32	32117	12.01.2026
When an existing staged user Joins my site, filled in user custom fields data isn't saved Bug	13	1082	28.06.2023
How to disable mail for staged users? Support	5	823	14.11.2024
Allow incoming email for automatic groups Feature groups , email-in	2	46	31.03.2026

IMAP синхронизация ставит нового пользователя в очередь, если Group.incoming_email не установлен в Group.email_username

Связанные темы