You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your improvement related to a problem? Please describe.
NServiceBus.Logging exists in all versions of Core. Its a logging abstraction by NServiceBus that is from the time the .NET ecosystem didn't have a defacto abstraction. Now that Microsoft.Extensions.Logging has matured there is no reason for NServiceBus users to use its abstraction.
This also fixes issues with NServiceBus.Logging.LogManager being a service locator pattern and any loggers created could conflict when multiple endpoints are hosted in a single process
Suggestion
Mark all public types in NServiceBus.Logging with [Obsolete]with guidance to transition toMicrosoft.Extensions.Logging`
Additional Context
No response
Related
Currently abstraction blocks certain refactorings like:
When we eventually will move away from our limited abstraction it would be nice if such an overview could be added (or even auto generated) in our docs.
Describe the suggested improvement
Is your improvement related to a problem? Please describe.
NServiceBus.Logging exists in all versions of Core. Its a logging abstraction by NServiceBus that is from the time the .NET ecosystem didn't have a defacto abstraction. Now that
Microsoft.Extensions.Logging
has matured there is no reason for NServiceBus users to use its abstraction.This also fixes issues with
NServiceBus.Logging.LogManager
being a service locator pattern and any loggers created could conflict when multiple endpoints are hosted in a single processSuggestion
Mark all public types in
NServiceBus.Logging
with [Obsolete]with guidance to transition to
Microsoft.Extensions.Logging`Additional Context
No response
Related
Currently abstraction blocks certain refactorings like:
AddNServiceBus
extension method onIServiceCollection
NServiceBus.Extensions.Hosting#519The text was updated successfully, but these errors were encountered: