تم تعطيل جميع رسائل البريد الإلكتروني بعد استعادة النسخة الاحتياطية؟

I wanted to do some testing with Discourse so I took a backup of my instance, wiped the DB (rm -fr /var/discourse/shared) and rebuilt the container.

I then created an admin user again and instead of following the setup wizard I went straight to admin, enabled restores, uploaded the backup and restored.

For whatever reason the outgoing email is disabled by an administrator message appeared.

I can confirm that the outgoing email was never disabled on the instance where the backup was made.

I did tick the option to enable readonly mode when performing backup operations.

Is this a bug or did I mess up somewhere?

إعجابَين (2)

يجب أن يظهر هذا الرسالة لفترة قصيرة فقط. من المفترض أن تختفي بعد انتهاء عملية الاستعادة، طالما لم يتم تعطيل البريد الإلكتروني الصادر عند إنشاء النسخة الاحتياطية. إذا لم تختفِ الإشعار تلقائيًا، فإن تحديث الصفحة في متصفحك قد يساعد.


أترك هذا كـ #contribute:bug، لأن الطريقة التي نعطل بها البريد الإلكتروني الصادر قبل الاستعادة عديمة الفائدة إلى حد كبير. يتم تعطيل Sidekiq أثناء الاستعادة، لذلك لا يمكن أن يحدث أي شيء خلال ذلك الوقت. وعملية الاستعادة تكتب فوق قيمة إعداد disable_email. يتطلب إصلاح هذا الخطأ تغيير إعداد الموقع بعد الاستعادة.

@eviltrout لقد أضفت هذا في When restoring a backup, disable emails. · discourse/discourse@ccf9b70 · GitHub. ما رأيك في هذا؟ هل يجب أن نصلح هذا الخطأ أم نزيل الكود؟ في رأيي، تغيير إعداد الموقع بعد الاستعادة سيكون غير متوقع تمامًا، لذا أنا أصوت لإزالته. بالإضافة إلى ذلك، لم يشك أحد في هذه المشكلة خلال ما يقرب من 3 سنوات… :wink:

3 إعجابات

I think the idea was that when a restore finishes the administrator might want to poke around and make sure everything is OK. Right now emails will start going out the second it finishes and the site might not be “approved.”

4 إعجابات

So therefore this is not a bug @gerhard

Maybe Discourse should make it a bit more clear to the admin (e.g. entry inserted in the backup log panel). The site setting for disable_emails is set to no while the message renders and this is confusing (admin panel says disable emails is off while the message on the top of the screen still says its disabled)

Well, I consider it a bug when a line of code doesn’t do what it’s supposed to do. :wink: Anyway, it’s fixed: FIX: Outgoing emails were not disabled after restoring backup · discourse/discourse@dc961fe · GitHub

I made a small adjustment, because I think it’s better to disable outgoing emails only for non-staff users instead of all of them.

7 إعجابات

This topic was automatically closed 24 hours after the last reply. New replies are no longer allowed.