starred/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2026-07-16 05:43:54 +03:00
Code Issues 822 Packages Projects Releases 17 Wiki Activity

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

New Issue
Closed
opened 2026-02-07 07:12:07 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-07 07:12:07 +03:00
Owner
Copy Link

📋 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>
OVERLORD added the pull-request label 2026-02-07 07:12:07 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
EFjellyfin.db
area:database
awaiting-feedback
backend
blocked
breaking change: web api
bug
build
ci
confirmed
discussion needed
dotnet future
downstream
duplicate
enhancement
feature
future
github-actions
good first issue
hdr
help wanted
invalid
investigation
librarydb
live-tv
lyrics
media playback
music
needs testing
nuget
performance
platform
pull-request
Mirrored from GitHub Pull Request
 question
regression
release critical
requires-web
roadmap
security
security
stale
support
syncplay
ui & ux
upstream
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13389
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?