[PR #14243] [MERGED] Rework startup topic handling and reenable output to logging framework #13898

Closed
opened 2026-02-07 07:20:59 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/14243
Author: @JPVenson
Created: 6/8/2025
Status: Merged
Merged: 6/9/2025
Merged by: @crobibero

Base: masterHead: bugfix/StartupLogfileAndLogOutput


📝 Commits (2)

  • 8dce8f9 Rework startup topic handling and reenable output to logging framework
  • 8acdf89 Rename file

📊 Changes

16 files changed (+255 additions, -60 deletions)

View changed files

📝 Jellyfin.Server/Migrations/JellyfinMigrationService.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateKeyframeData.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateLibraryDbCompatibilityCheck.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MigrateRatingLevels.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MoveExtractedFiles.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Routines/MoveTrickplayFiles.cs (+1 -1)
📝 Jellyfin.Server/Migrations/Stages/CodeMigration.cs (+14 -4)
📝 Jellyfin.Server/Program.cs (+7 -4)
📝 Jellyfin.Server/ServerSetupApp/IStartupLogger.cs (+42 -1)
📝 Jellyfin.Server/ServerSetupApp/SetupServer.cs (+3 -14)
Jellyfin.Server/ServerSetupApp/StartupLogTopic.cs (+31 -0)
📝 Jellyfin.Server/ServerSetupApp/StartupLogger.cs (+50 -28)
Jellyfin.Server/ServerSetupApp/StartupLoggerExtensions.cs (+18 -0)
Jellyfin.Server/ServerSetupApp/StartupLoggerOfCategory.cs (+56 -0)
📝 tests/Jellyfin.Server.Integration.Tests/JellyfinApplicationFactory.cs (+27 -2)

📄 Description

Startup log entries are now ensured to be written to logfile again


🔄 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/14243 **Author:** [@JPVenson](https://github.com/JPVenson) **Created:** 6/8/2025 **Status:** ✅ Merged **Merged:** 6/9/2025 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `bugfix/StartupLogfileAndLogOutput` --- ### 📝 Commits (2) - [`8dce8f9`](https://github.com/jellyfin/jellyfin/commit/8dce8f9952dd184e5db6fecadb1d30004208db59) Rework startup topic handling and reenable output to logging framework - [`8acdf89`](https://github.com/jellyfin/jellyfin/commit/8acdf89fe893c0fbeff6d40d706e62efd00cc81d) Rename file ### 📊 Changes **16 files changed** (+255 additions, -60 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Server/Migrations/JellyfinMigrationService.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateKeyframeData.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateLibraryDbCompatibilityCheck.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MigrateRatingLevels.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MoveExtractedFiles.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Routines/MoveTrickplayFiles.cs` (+1 -1) 📝 `Jellyfin.Server/Migrations/Stages/CodeMigration.cs` (+14 -4) 📝 `Jellyfin.Server/Program.cs` (+7 -4) 📝 `Jellyfin.Server/ServerSetupApp/IStartupLogger.cs` (+42 -1) 📝 `Jellyfin.Server/ServerSetupApp/SetupServer.cs` (+3 -14) ➕ `Jellyfin.Server/ServerSetupApp/StartupLogTopic.cs` (+31 -0) 📝 `Jellyfin.Server/ServerSetupApp/StartupLogger.cs` (+50 -28) ➕ `Jellyfin.Server/ServerSetupApp/StartupLoggerExtensions.cs` (+18 -0) ➕ `Jellyfin.Server/ServerSetupApp/StartupLoggerOfCategory.cs` (+56 -0) 📝 `tests/Jellyfin.Server.Integration.Tests/JellyfinApplicationFactory.cs` (+27 -2) </details> ### 📄 Description Startup log entries are now ensured to be written to logfile again --- <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 07:20:59 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13898