[PR #2071] [MERGED] add filesize limit for logfiles and a maximum logfile count #8988

New Issue

OVERLORD · 2026-02-07T05:54:05+03:00

OVERLORD commented

2026-02-07 05:54:05 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2071
Author: @excelite
Created: 11/28/2019
Status: ✅ Merged
Merged: 11/29/2019
Merged by: @JustAMan

Base: master ← Head: add_default_values_to_logconfig

📝 Commits (1)

6069134 added filesize limit for logfiles and a maximum logfile count

📊 Changes

1 file changed (+3 additions, -0 deletions)

View changed files

📝 Jellyfin.Server/Resources/Configuration/logging.json (+3 -0)

📄 Description

Changes
Adds default values to logger config by using the loggers configuration arguments.

Issues
Fixes #2056
Fixes #2069

Detailed description
This PR adds the following default values to the shipped logging configuration:

100MB filesize limit
activates filesize based logrotation
sets the number of retained logfiles to three

The number of the retained logfiles is derived from the default value of the logfile cleanup task that sets the age of logfiles to three days. Since the time based logfile rotation is set to "Days" interval this would result in three logfiles to keep.

The default logfile size was chosen to ensure that limited memory devices like a raspberry can handle the logfile size with still sufficient ressources left for other processes.
Looking at personal logfile sizes of a 5 user instance with multiple day-to-day usage files range between 10MB to 30MB), a default value of 100MB should cover a large percentage of the userbase without adjustments.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/2071 **Author:** [@excelite](https://github.com/excelite) **Created:** 11/28/2019 **Status:** ✅ Merged **Merged:** 11/29/2019 **Merged by:** [@JustAMan](https://github.com/JustAMan) **Base:** `master` ← **Head:** `add_default_values_to_logconfig` --- ### 📝 Commits (1) - [`6069134`](https://github.com/jellyfin/jellyfin/commit/60691349a11f541958e0b2247c9abc13cb40c9fb) added filesize limit for logfiles and a maximum logfile count ### 📊 Changes **1 file changed** (+3 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Server/Resources/Configuration/logging.json` (+3 -0) </details> ### 📄 Description **Changes** Adds default values to logger config by using the loggers configuration arguments. **Issues** Fixes #2056 Fixes #2069 **Detailed description** This PR adds the following default values to the shipped logging configuration: - 100MB filesize limit - activates filesize based logrotation - sets the number of retained logfiles to three The number of the retained logfiles is derived from the default value of the logfile cleanup task that sets the age of logfiles to three days. Since the time based logfile rotation is set to "Days" interval this would result in three logfiles to keep. The default logfile size was chosen to ensure that limited memory devices like a raspberry can handle the logfile size with still sufficient ressources left for other processes. Looking at personal logfile sizes of a 5 user instance with multiple day-to-day usage files range between 10MB to 30MB), a default value of 100MB should cover a large percentage of the userbase without adjustments. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-07 05:54:05 +03:00

OVERLORD closed this issue

2026-02-07 05:54:08 +03:00

Sign in to join this conversation.

Branches Tags

master

release-10.11.z

renovate/skiasharp-monorepo

JPVenson-patch-1

renovate/morestachio-5.x

release-10.9.z

openapi-cache

release-10.10.z

explicit-load

explicit-load-v2

feature/entity-framework

only-count-folder-item

fix/ci-workflow-refactor

release-10.8.z

release-10.7.z

EraYaN-add-management-interface

release-10.6.z

release-10.5.z

release-10.4.z

release-10.3.z

release-10.2.z

release-10.1.0

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#8988