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 #15681] [MERGED] Fix symlinked file size #14384

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/15681
Author: @ivanjx
Created: 12/1/2025
Status: ✅ Merged
Merged: 12/4/2025
Merged by: @crobibero

Base: release-10.11.z ← Head: symlink-fix

📝 Commits (4)

  • bdff13a Fix symlink file size reading
  • 3650bdd Resolve symlinked file path before passing into PhysicalFile
  • b271ef7 Revert "Fix symlink file size reading"
  • 8310aea Keep filename the same as symlinked path

📊 Changes

1 file changed (+13 additions, -1 deletions)

View changed files

📝 Jellyfin.Api/Controllers/LibraryController.cs (+13 -1)

📄 Description

Changes
this pr resolves real file path of symlinks before passing it into PhysicalFile. without this, PhysicalFile would get a symlinked path and returns the file size of the symlink itself rather than the actual file it points to.

Issues

Fixes #15279
Fixes #15295

🔄 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/15681 **Author:** [@ivanjx](https://github.com/ivanjx) **Created:** 12/1/2025 **Status:** ✅ Merged **Merged:** 12/4/2025 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `release-10.11.z` ← **Head:** `symlink-fix` --- ### 📝 Commits (4) - [`bdff13a`](https://github.com/jellyfin/jellyfin/commit/bdff13a05a0753d2e6e61fe5c66fac5ed9903caa) Fix symlink file size reading - [`3650bdd`](https://github.com/jellyfin/jellyfin/commit/3650bddbf0ce63937975016226c769c78cb87822) Resolve symlinked file path before passing into PhysicalFile - [`b271ef7`](https://github.com/jellyfin/jellyfin/commit/b271ef713016a732dca798c92be1cf64f5f721c2) Revert "Fix symlink file size reading" - [`8310aea`](https://github.com/jellyfin/jellyfin/commit/8310aead601c34bf9b0b912a26a2b3d9c0eaa02a) Keep filename the same as symlinked path ### 📊 Changes **1 file changed** (+13 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Api/Controllers/LibraryController.cs` (+13 -1) </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** this pr resolves real file path of symlinks before passing it into `PhysicalFile`. without this, `PhysicalFile` would get a symlinked path and returns the file size of the symlink itself rather than the actual file it points to. **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> Fixes #15279 Fixes #15295 --- <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:29:00 +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#14384
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?