mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-07-16 05:43:54 +03:00
[PR #15752] [MERGED] Fix case sensitivity edge case #14411
Reference in New Issue
Block a user
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?
📋 Pull Request Information
Original PR: https://github.com/jellyfin/jellyfin/pull/15752
Author: @Collin-Swish
Created: 12/9/2025
Status: ✅ Merged
Merged: 12/12/2025
Merged by: @Bond-009
Base:
release-10.11.z← Head:fix-name-case-insensitivity📝 Commits (1)
2e8d9a3Fix case sensitivity edge case📊 Changes
1 file changed (+1 additions, -1 deletions)
View changed files
📝
Jellyfin.Server.Implementations/Users/UserManager.cs(+1 -1)📄 Description
Summary
Both UserController.UpdateUser and UserManager.RenameUser compare the new username with the old username. UpdateUser checked for case sensitive inequality while RenameUser was case insensitive. This led to an edge case where the username change would lead to an incomplete request if the only difference was the character case. e.g collin to Collin. This left the client unresponsive while it waited for a server request that would never come.
Changes
Changed equality in UserManager.RenameUser from case insensitive to case sensitive.
Issues
Fixes #15417
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.