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.
Это сообщение должно отображаться только в течение короткого времени. Оно должно исчезнуть после завершения восстановления, при условии, что исходящая почта не была отключена в момент создания резервной копии. Если уведомление не исчезает автоматически, попробуйте обновить страницу в браузере.
Оставляю это как Contribute > Bug, потому что текущий способ отключения исходящей почты перед восстановлением совершенно бесполезен. Sidekiq отключается во время восстановления, поэтому в этот период ничего не происходит. Кроме того, процесс восстановления перезаписывает значение настройки disable_email. Исправление бага потребовало бы изменения настройки сайта после завершения восстановления.
@eviltrout Вы добавили этот код в When restoring a backup, disable emails. · discourse/discourse@ccf9b70 · GitHub. Каково ваше мнение? Стоит ли исправлять баг или удалить код? На мой взгляд, изменение настройки сайта после восстановления было бы довольно неожиданным, поэтому я выступаю за удаление. Более того, за почти 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.”
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)
