IMAP同期ステージ：Group.incoming_emailがGroup.email_usernameに設定されていない新規ユーザー

thoka · 2021 年 2 月 6 日午後 6: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 日午後 8:08

IMAPはまだ少し開発途中でしょうか、@sam？

sam · 2021 年 2 月 8 日午前 1:00

Meta の本番環境でも実際に使用しており、十分にサポートされているはずです。

@martin さん、このレポートを少し確認してもらえますか？

martin · 2021 年 2 月 8 日午前 3: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 日午前 1:30

これは FEATURE: Use group SMTP settings for sending user notification emails (initial) by martin-brennan · Pull Request #13220 · discourse/discourse · GitHub で修正され、グループのメール設定 UI も最近大幅に刷新されました。ただし、IMAP は依然としてアルファ版の段階にあるため、その機能を使用する場合は自己責任となります。

トピック		返信	表示
Email into group sends empty notification Bug	12	2003	2016 年 1 月 20 日
IMAP support for group inboxes Announcements new-feature	32	32087	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に設定されていない新規ユーザー

関連トピック