search

rebus-org / Rebus.AzureServiceBus

:bus: Azure Service Bus transport for Rebus
https://mookid.dk/category/rebus
Other
33 stars 20 forks source link

WIP refactor(transport): separate management from transport #89

Closed riezebosch closed 1 year ago

riezebosch commented 1 year ago

First attempt to separate the infrastructure logic from the transport.

Would've been nice when ITransport itself was segregated into at least 2 interfaces. Also, the IInitialize doesn't seem to be invoked automatically for anything other than the transport.

Maybe you can validate if this split makes sense. There's still the GetActualAddress where I didn't really have a good idea where it belongs. It would remove the NameFormatter dependency from the Transport but not sure that it's better and have an additional dependency on the infra/management.

Rebus is MIT-licensed. The code submitted in this pull request needs to carry the MIT license too. By leaving this text in, I hereby acknowledge that the code submitted in the pull request has the MIT license and can be merged with the Rebus codebase.

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.