Update Changelog, log restarts

btcpayserver · Jan 19, 2023 · 5f24b41 · 5f24b41
1 parent b577c0a
commit 5f24b41
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/BTCPayServer/Controllers/UIServerController.cs b/BTCPayServer/Controllers/UIServerController.cs
@@ -133,7 +133,6 @@ public IActionResult Maintenance()
         public async Task<IActionResult> Maintenance(MaintenanceViewModel vm, string command)
         {
             vm.CanUseSSH = _sshState.CanUseSSH;
-
             if (command != "soft-restart" && !vm.CanUseSSH)
             {
                 TempData[WellKnownTempData.ErrorMessage] = "Maintenance feature requires access to SSH properly configured in BTCPay Server configuration.";
@@ -221,12 +220,14 @@ public async Task<IActionResult> Maintenance(MaintenanceViewModel vm, string com
                 var error = await RunSSH(vm, $"btcpay-restart.sh");
                 if (error != null)
                     return error;
+                Logs.PayServer.LogInformation("A hard restart has been requested");
                 TempData[WellKnownTempData.SuccessMessage] = $"BTCPay will restart momentarily.";
             }
             else if (command == "soft-restart")
             {
                 TempData[WellKnownTempData.SuccessMessage] = $"BTCPay will restart momentarily.";
-                ApplicationLifetime.StopApplication();
+                Logs.PayServer.LogInformation("A soft restart has been requested");
+                _ = Task.Delay(3000).ContinueWith((t) => ApplicationLifetime.StopApplication());
             }
             else
             {

diff --git a/Changelog.md b/Changelog.md
@@ -2,6 +2,11 @@
 
 ## 1.7.4
 
+Note for integrators such as Raspiblitz or Umbrel: As part of our effort to make BTCPay Server more welcoming to plugins, we have made a change that may impact you.
+
+Previously, when a user uninstalled or installed a new plugin, BTCPay Server would prompt them to restart the server by clicking on a button. Prior to version 1.7.4, this restart button was not functional due to being coupled to our own Docker deployment stack.
+
+As of now, the restart button will instead terminate the BTCPay Server process. The process manager, such as systemd or docker should then automatically restart BTCPay Server. Please ensure that automatic restart capability exists.
 ### Bug fixes
 
 * Fix LNURL authentication as 2FA method (#4501) @dennisreimann