#254: Do not delete default rule if no other rule exists

zarusz / SlimMessageBus

Lightweight message bus interface for .NET (pub/sub and request-response) with transport plugins for popular message brokers.

Apache License 2.0

467 stars 78 forks source link

#254: Do not delete default rule if no other rule exists #255

Closed EtherZa closed 4 months ago

EtherZa commented 4 months ago

Bug in #245 resulted in the default rule of a subscription being deleted on creation regardless as to where there was a rule to replace it or not.

PR returns to the original flow where the default rule will only be deleted if the subscription has just been created and there are alternate rules to apply.

zarusz commented 4 months ago

@EtherZa I think some more checks are missing further down and the int test is failing. However, I've also made progress on my own fix in #257. Can we use that other PR for the resolution of the bug, please? There is more extras I've added on the way.