-
Notifications
You must be signed in to change notification settings - Fork 870
Open
Description
Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
- I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
- I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue
Is your feature request related to a problem? Please describe.
Nowadays closelog() has issues yet the reason for it was the nested openlog() calls which are impossible to track in spaghetti code and across legacy/MVC. The global nature of these calls and eventually missing context (because a subroutine changed the logging) makes this impossible to handle for multiple local log locations.
Describe the solution you like
Write a class-based wrapper that can handle the log context switches if needed and use it everywhere.
Describe alternatives you considered
N/A
Additional context
N/A
Metadata
Metadata
Assignees
Labels
featureAdding new functionalityAdding new functionality