Dnn.Platform icon indicating copy to clipboard operation
Dnn.Platform copied to clipboard

[Enhancement]: Change from Log4net to Serilog

Open donker opened this issue 1 year ago • 1 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Description of problem

DNN Currently uses Log4net for logging. It is also included as a DNN branded fork of log4net which makes us vulnerable. Although log4net has served as well over the years, there are newer solutions that provide us with features log4net cannot offer.

Note this issue is a continuation of issue #2801

Description of solution

The solution is to deprecate the old log4net library included with DNN and move to (non-forked) Serilog. Serilog offers many more sinks and can be configured to clean up the log directory. It would also provide an opportunity to move the DotNetNuke.Instrumentation library to .net standard.

Description of alternatives considered

The alternative considered is keeping log4net and updating the existing code to keep in sync with the original project. Because our code includes CodeCop fixes this will be a time-consuming process.

Anything else?

No response

Do you be plan to contribute code for this enhancement?

  • [X] Yes

Would you be interested in sponsoring this enhancement?

  • [ ] Yes

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

donker avatar Nov 01 '24 19:11 donker

THanks @donker for bringing this back!

mitchelsellers avatar Nov 01 '24 20:11 mitchelsellers