Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added required nuget package to enable serilog based centralized logging in gelf format #2082

Merged
merged 1 commit into from
Dec 3, 2019

Conversation

excelite
Copy link

Changes

  • added the nuget package to enable Serilog's GELF logging.

Issues
none

feature requests
Add support for centralized logging

example configuration
/config/logging.json

{
    "Serilog": {
        "MinimumLevel": "Information",
        "WriteTo": [
            {
                "Name": "Console",
                "Args": {
                    "outputTemplate": "[{Timestamp:HH:mm:ss}] [{Level:u3}] {Message:lj}{NewLine}{Exception}"
                }
            },
            {
                "Name": "Graylog",
                "Args": {
                    "HostnameOrAddress": "localhost",
                    "Port": "12202",
                    "TransportType": "Udp"
                }
            }
        ]
    }
}

alternative valid values:
HostnameOrAddress can also be an IP Address like:
"192.0.0.1"
TransportType can also be one of these:
"Tcp", "Http"

background
The logging in Jellyfin is quite messy (read: non-uniform). This includes two Jellyfin modules logging to .txt files and the others to .log files. Also some modules use the more advanced way of enriching the log messages in a more "structured data" like way, while others don't.

I think it would be a nice way to start a cleanup of the logger with easy way to evaluate the results.

@Bond-009 Bond-009 merged commit 01c5dce into jellyfin:master Dec 3, 2019
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.

None yet

3 participants