search

dnnsoftware / Dnn.Platform

DNN (formerly DotNetNuke) is the leading open source web content management platform (CMS) in the Microsoft ecosystem.
https://dnncommunity.org/
MIT License
1.03k stars 751 forks source link

[Enhancement]: Change from Log4net to Serilog #6181

Open donker opened 11 hours ago

donker commented 11 hours ago

Is there an existing issue for this?

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?

Would you be interested in sponsoring this enhancement?

Code of Conduct

mitchelsellers commented 11 hours ago

THanks @donker for bringing this back!