[PR #12832] [MERGED] Check for path overlaps #13208

New issue

Closed

opened 2025-12-22 09:44:02 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 09:44:02 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/12832
Author: @JPVenson
Created: 10/16/2024
Status: ✅ Merged
Merged: 4/26/2025
Merged by: @crobibero

Base: master ← Head: feature/mount_markerfiles

📝 Commits (10+)

e88240c Check for path overlaps
e8ab3c5 Added recursive check for cache and transcode cache
804135a Add option to disable folder corss check.
364ff39 Fixed Tests
c50525c removed option to skip overlap check
c5eefb3 Merge branch 'jellyfin:master' into feature/mount_markerfiles
2e50e63 Merge branch 'master' into feature/mount_markerfiles
f220bb8 Apply suggestions from code review
06d6113 Update IApplicationPaths.cs
284308f Update IApplicationPaths.cs

📊 Changes

8 files changed (+71 additions, -5 deletions)

View changed files

📝 Emby.Server.Implementations/AppBase/BaseApplicationPaths.cs (+44 -1)
📝 Emby.Server.Implementations/AppBase/BaseConfigurationManager.cs (+1 -0)
📝 Emby.Server.Implementations/Library/LibraryManager.cs (+0 -2)
📝 Emby.Server.Implementations/ServerApplicationPaths.cs (+7 -0)
📝 Jellyfin.Server/Program.cs (+1 -0)
📝 MediaBrowser.Common/Configuration/EncodingConfigurationExtensions.cs (+1 -2)
📝 MediaBrowser.Common/Configuration/IApplicationPaths.cs (+13 -0)
📝 MediaBrowser.Controller/IServerApplicationPaths.cs (+4 -0)

📄 Description

Added marker file to identify most build in folders and check for cross mounting folders to then stop server.

_{🔄 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/12832 **Author:** [@JPVenson](https://github.com/JPVenson) **Created:** 10/16/2024 **Status:** ✅ Merged **Merged:** 4/26/2025 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `feature/mount_markerfiles` --- ### 📝 Commits (10+) - [`e88240c`](https://github.com/jellyfin/jellyfin/commit/e88240c61c44e163005ad878e1aa5d1bf4edd3d2) Check for path overlaps - [`e8ab3c5`](https://github.com/jellyfin/jellyfin/commit/e8ab3c58d302248fdf0c1a473e6e589d0d6e300e) Added recursive check for cache and transcode cache - [`804135a`](https://github.com/jellyfin/jellyfin/commit/804135aad2e664b95a93fa6118a931ebdf9c568f) Add option to disable folder corss check. - [`364ff39`](https://github.com/jellyfin/jellyfin/commit/364ff39df9fa77c148051cb3422b648bb88ab086) Fixed Tests - [`c50525c`](https://github.com/jellyfin/jellyfin/commit/c50525c57145a688890f89cc22d6466b8df8f16e) removed option to skip overlap check - [`c5eefb3`](https://github.com/jellyfin/jellyfin/commit/c5eefb327721367beca3e4c595da3b0c4079064c) Merge branch 'jellyfin:master' into feature/mount_markerfiles - [`2e50e63`](https://github.com/jellyfin/jellyfin/commit/2e50e6344c84428b41f5649385d07ce4b2645de0) Merge branch 'master' into feature/mount_markerfiles - [`f220bb8`](https://github.com/jellyfin/jellyfin/commit/f220bb85fd8df720a33dfcabadf9f63c3e9c0f78) Apply suggestions from code review - [`06d6113`](https://github.com/jellyfin/jellyfin/commit/06d611353e9d35020c64a1df96457c0f2ad6eb34) Update IApplicationPaths.cs - [`284308f`](https://github.com/jellyfin/jellyfin/commit/284308ff26b38a03e9a79e71f45965c8e94faebb) Update IApplicationPaths.cs ### 📊 Changes **8 files changed** (+71 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/AppBase/BaseApplicationPaths.cs` (+44 -1) 📝 `Emby.Server.Implementations/AppBase/BaseConfigurationManager.cs` (+1 -0) 📝 `Emby.Server.Implementations/Library/LibraryManager.cs` (+0 -2) 📝 `Emby.Server.Implementations/ServerApplicationPaths.cs` (+7 -0) 📝 `Jellyfin.Server/Program.cs` (+1 -0) 📝 `MediaBrowser.Common/Configuration/EncodingConfigurationExtensions.cs` (+1 -2) 📝 `MediaBrowser.Common/Configuration/IApplicationPaths.cs` (+13 -0) 📝 `MediaBrowser.Controller/IServerApplicationPaths.cs` (+4 -0) </details> ### 📄 Description Added marker file to identify most build in folders and check for cross mounting folders to then stop server. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 09:44:02 +01:00

closed this issue
added the
pull-request
label

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#13208

No description provided.

Rows
Columns