Skip to content

Conversation

@lineker
Copy link

@lineker lineker commented Aug 17, 2015

Hi there,

I removed the strong reference to the log4net library and added an interface so, it can be implemented with any library.

Let me know if you have any questions.

Lineker Tomazeli added 4 commits August 17, 2015 14:51
Abstracted the logger interface so different log libraries can be used.
Implemented a default logger that print out using Diagnostic.Trace
Implemented ILogger interface on JwtAuthForWebAPI .SampleSite
Abstracted the logger interface so different log libraries can be used.
Implemented a default logger that print out using Diagnostic.Trace
Implemented ILogger interface on JwtAuthForWebAPI .SampleSite
@jamiekurtz
Copy link
Owner

@lineker so sorry... I just saw this PR. Not sure what happened to the initial email from Github... regardless, do you still need this merged?

@bastianeicher
Copy link

I'd also like to see this merged. We use NLog in most of our web services so avoiding a log4net reference would be great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants