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

New issue

Closed

opened 2025-12-22 07:08:23 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 07:08:23 +01:00

Owner

📋 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>

backuprepo

2025-12-22 07:08:23 +01:00

closed this issue
added the
pull-request
label

backuprepo referenced this issue

2025-12-22 08:53:34 +01:00

[PR #8812] [MERGED] Validate requested user id #11685

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#8812

No description provided.

Rows
Columns