IMAP同步阶段：如果Group.incoming_email未设置为Group.email_username，则为新用户

thoka · 2021 年2 月 6 日 18:18

在尝试 IMAP 同步时，从 IMAP 账户导入过程中为 Group.email_username 创建了一个临时用户。

在尝试理解发生了什么时，我注意到 Group.find_by_email 并未考虑 Group.email_username，因此至少需要设置 Group.incoming_email 以包含该值。

复现步骤：

启用 IMAP 同步
在群组上配置电子邮件同步选项，但保持 Group.incoming_email 未配置
向 Group.email_username 发送邮件

codinghorror · 2021 年2 月 7 日 20:08

我认为 IMAP 目前仍有一些工作需要完善，@sam？

sam · 2021 年2 月 8 日 01:00

它应该得到了合理的支持，我们在 Meta 的生产环境中确实使用了它。

@martin 你能快速看一下这份报告吗？

martin · 2021 年2 月 8 日 03:44

感谢 @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 · 2021 年6 月 11 日 01:30

该问题已通过 FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub 修复，且群组电子邮件配置界面近期已全面升级。不过，IMAP 功能仍处于 Alpha 阶段，请谨慎使用，风险自负。

话题		回复	浏览量
Email into group sends empty notification Bug	12	2003	2016 年1 月 20 日
IMAP support for group inboxes Announcements new-feature	32	32088	2026 年1 月 12 日
When an existing staged user Joins my site, filled in user custom fields data isn't saved Bug	13	1070	2023 年6 月 28 日
How to disable mail for staged users? Support	5	821	2024 年11 月 14 日
Allow incoming email for automatic groups Feature groups , email-in	2	30	2026 年3 月 31 日

IMAP同步阶段：如果Group.incoming_email未设置为Group.email_username，则为新用户

相关话题