هل يتطلب تصنيف البريد الإلكتروني تمكين الرد عبر البريد الإلكتروني؟

الموقف

  • تم تعطيل reply_by_email_enabled، نحن لا نريد أن يرد الأشخاص عبر البريد الإلكتروني
  • فئة مخصصة للمسؤولين فقط للإشعارات الواردة من نظام خارجي
  • الفئة لديها عنوان بريد إلكتروني مُعد
  • تم تمكين email_in
  • يتم إرسال رسائل البريد الإلكتروني إلى عنوان البريد الإلكتروني للفئة
  • ترتد رسائل البريد الإلكتروني مع الخطأ BadDestinationAddress

السبب

بعد ساعة من تصحيح الأخطاء، صادفت ما يلي FIX: Disallow replies to categories when reply by email disabled (#33… · discourse/discourse@e05ef50 · GitHub

إصلاح: منع الردود على الفئات عند تعطيل الرد عبر البريد الإلكتروني (#33641)

عندما يتم تعيين الإعداد reply_by_email_enabled على خطأ، فإننا لم نعد ندرج رابط الرد في إشعارات البريد الإلكتروني. ومع ذلك، فإننا لا نمنع رسائل البريد الإلكتروني الفعلية المرسلة إلى عنوان email_in مُعد مرتبط بفئة. هذا التغيير يأخذ في الاعتبار الإعداد في Email::Receiver#check_address.

والرمز المتأثر affected code لن يُرجع فئة بعد الآن إذا لم يكن reply_by_email_enabled ممكّنًا.

    def self.check_address(address, include_verp = false)
      # فقط تحقق من مجموعة/فئة عندما يكون 'email_in' ممكّنًا
      if SiteSetting.email_in
        group = Group.find_by_email(address)
        return group if group

        category = Category.find_by_email(address)
        return category if category && SiteSetting.reply_by_email_enabled?  #-- تمت الإضافة
      end

لماذا؟ :scream:

لدي الكثير من الأسئلة:

  1. لماذا هذا التغيير؟ على الإطلاق؟ إنه يجعل النظام أقل مرونة فقط. إذا كنت لا أريد أن يرسل الناس بريدًا إلكترونيًا إلى فئة، فسأقوم ببساطة بإزالة عنوان البريد الإلكتروني من الفئة؟
  2. لماذا تم تنفيذ ذلك بهذه الطريقة؟
  • على ما يبدو أن البريد الإلكتروني الوارد إلى المجموعات ليس مشكلة؟
  • إذا كان reply_by_email_enabled خطأ، فلا حاجة للتكرار عبر الفئات؟
  • عن طريق إزالة الفئة من قائمة الفئات، يصبح الخطأ BadDestinationAddress وهو خطأ خاطئ حقًا ويصعب جدًا تصحيحه
  1. الإعداد الذي أصبح يعتمد عليه فجأة يسمى الرد عبر البريد الإلكتروني. هذا ليس ما أفعله.

بصرف النظر عن الطريقة التي تم بها تنفيذ هذا، أفشل بصراحة في رؤية النقطة.

لا يمكنني التفكير في أي موقف حيث أن مجرد إزالة عنوان البريد الإلكتروني من الفئة سيؤدي الغرض إذا أراد المرء منع الناس من إرسال بريد إلكتروني إلى فئة. والنتيجة المترتبة على ذلك هي أنه أصبح من المستحيل الآن أن تحتوي فئة على رسائل بريد إلكتروني واردة دون تمكين الرد عبر البريد الإلكتروني على مستوى الموقع.

إذا كان هناك سبب وجيه فاتني، فلا تتردد في إعادة تصنيف هذا على أنه Feature

4 إعجابات

لقد اكتشفت هذا بنفسي وأنا أبحث فيه بالفعل.

أتفق على أن الإصلاح الذي تم في FIX: Disallow replies to categories when reply by email disabled (#33… · discourse/discourse@e05ef50 · GitHub كان سيئًا لأنه اعتمد على وأدخل تأثيرًا جانبيًا، لكننا لم نلاحظ ذلك لأن الاختبارات مرتبة بطريقة لا تتوفر فيها الشروط اللازمة لتشغيله. أنا أعمل على تصحيح ذلك الآن.

إعجابَين (2)