flywithu/CryptoExchange.Net
1
0
Fork 0
mirror of https://github.com/JKorf/CryptoExchange.Net.git synced 2026-10-14 07:01:46 +00:00
Code Issues Releases Wiki Activity

Fix for duplicate subscription coverage

Browse Source
This commit is contained in:
Jkorf
2026-10-13 10:37:49 +02:00
parent 4f25a6e7d6
commit 5135047097
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
CryptoExchange.Net/Sockets/Default/SocketConnection.cs
+2 -2
View File
@@ -1076,7 +1076,7 @@ namespace CryptoExchange.Net.Sockets.Default
=> _listeners.OfType<Subscription>().Any(
x => x != subscription
&& x.Active
&& x.MessageRouter.Routes.All(l => subscription.MessageRouter.ContainsCheck(l)));
&& subscription.MessageRouter.Routes.All(l => x.MessageRouter.ContainsCheck(l)));
/// <summary>
/// Whether there is a subscription earlier in the list covering all the routes the subscription lists
@@ -1088,7 +1088,7 @@ namespace CryptoExchange.Net.Sockets.Default
if (other == subscription)
return false;
if (other.Active && other.MessageRouter.Routes.All(x => subscription.MessageRouter.ContainsCheck(x)))
if (other.Active && subscription.MessageRouter.Routes.All(x => other.MessageRouter.ContainsCheck(x)))
return true;
}