Is this justified?
MODIFY
WalletWasabi/BitcoinCore/Processes/BitcoindRpcProcessBridge.cs
WalletWasabi/BitcoinCore/Processes/BitcoindRpcProcessBridge.cs
@@ -128,9 +128,6 @@ namespace WalletWasabi.BitcoinCore.Processes
ProcessAsync process = Process; // process is guaranteed to be non-null at this point.
- var rpcRan = false;
- try
- {
using var cts = new CancellationTokenSource(ReasonableCoreShutdownTimeout);
int? pid = await PidFile.TryReadAsync().ConfigureAwait(false);
@@ -144,7 +141,6 @@ namespace WalletWasabi.BitcoinCore.Processes
try
{
await RpcClient.StopAsync().ConfigureAwait(false);
- rpcRan = true;
}
catch (Exception ex)
{
@@ -167,14 +163,6 @@ namespace WalletWasabi.BitcoinCore.Processes
{
Logger.LogDebug("User is NOT responsible for the daemon process.");
}
- }
- catch
- {
- if (!onlyOwned && !rpcRan)
- {
- await RpcClient.StopAsync().ConfigureAwait(false);
- }
- }
Logger.LogDebug("<");
}

Is this justified?