TorControlClient: Bug in parameters of SETEVENTS.
MODIFY
WalletWasabi.Tests/UnitTests/Tor/Control/TorControlClientTests.cs
WalletWasabi.Tests/UnitTests/Tor/Control/TorControlClientTests.cs
@@ -187,12 +187,12 @@ namespace WalletWasabi.Tests.UnitTests.Tor.Control
Task task = client.SubscribeEventsAsync(new string[] { "CIRC", "STATUS_CLIENT" }, timeoutCts.Token);
// CIRC is already subscribed.
- Logger.LogTrace("Server: Wait for 'SETEVENTS CIRC,STATUS_CLIENT' command.");
+ Logger.LogTrace("Server: Wait for 'SETEVENTS CIRC STATUS_CLIENT' command.");
string command = await toServer.Reader.ReadLineAsync(timeoutCts.Token);
// This means that BOTH 'CIRC' and 'STATUS_CLIENT' must be subscribed now.
// Note: Given we count logical event subscriptions, 'CIRC' is now (logically) subscribed twice!
- Assert.Equal("SETEVENTS CIRC,STATUS_CLIENT", command);
+ Assert.Equal("SETEVENTS CIRC STATUS_CLIENT", command);
Logger.LogTrace("Server: Reply with OK code.");
await toClient.Writer.WriteAsciiAndFlushAsync("250 OK\r\n", timeoutCts.Token);

MODIFY
WalletWasabi/Tor/Control/TorControlClient.cs
WalletWasabi/Tor/Control/TorControlClient.cs
@@ -259,7 +259,7 @@ namespace WalletWasabi.Tor.Control
}
// Get all event names that must be subscribed.
- subscribedEventNames = string.Join(',', SubscribedEvents.Keys);
+ subscribedEventNames = string.Join(' ', SubscribedEvents.Keys);
}
if (sendCommand)
@@ -307,7 +307,7 @@ namespace WalletWasabi.Tor.Control
}
// Get all event names that remained.
- subscribedEventNames = string.Join(',', SubscribedEvents.Keys);
+ subscribedEventNames = string.Join(' ', SubscribedEvents.Keys);
}
if (sendCommand)

TorControlClient: Bug in parameters of SETEVENTS.