NLog reference in Solace.Messaging.Trace.Propagation

PubSub+ Event Broker
1

Hi, you are referencing directly NLog 5.2.0 in Solace.Messaging.Trace.Propagation .NET package and using its type Logger as a base type for SolLogger.

Any chance to remove NLog dependency and use ILogger instead?

Problem is that we cant link it because we use different version of NLog and in general it is good practice to not pull this kind of dependencies on specific logger into general purpose library.

thanks,

Boris

2

Hi @borisbucha ,

Hope you’re doing well! Just a heads up that we saw this and are looking into it. Will get back to you soon!

3

awesome, thanks @marc.dipasquale

Also, opensourcing these client libs would be great :slight_smile:

Not sure how realistic.

We’d happy send PRs for small changes like these.