[PR #1926] [MERGED] Add clearer exceptions, warnings and docs #8951

Closed
opened 2026-02-07 05:53:22 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/1926
Author: @Bond-009
Created: 10/20/2019
Status: Merged
Merged: 11/22/2019
Merged by: @dkanada

Base: masterHead: auth


📝 Commits (4)

  • fef35d0 Add clearer exceptions, warnings and docs
  • 24c076e Update Emby.Server.Implementations/Library/UserViewManager.cs
  • 2b1e766 Update MediaBrowser.Controller/Library/IUserManager.cs
  • d7335f6 Reword exception message

📊 Changes

10 files changed (+63 additions, -57 deletions)

View changed files

📝 Emby.Server.Implementations/Cryptography/CryptographyProvider.cs (+3 -7)
📝 Emby.Server.Implementations/Library/DefaultAuthenticationProvider.cs (+4 -1)
📝 Emby.Server.Implementations/Library/UserManager.cs (+6 -14)
📝 Emby.Server.Implementations/Library/UserViewManager.cs (+10 -10)
📝 Emby.Server.Implementations/Session/SessionManager.cs (+14 -13)
📝 MediaBrowser.Api/UserLibrary/UserViewsService.cs (+7 -1)
📝 MediaBrowser.Common/Cryptography/PasswordHash.cs (+3 -3)
📝 MediaBrowser.Controller/Entities/Folder.cs (+9 -1)
📝 MediaBrowser.Controller/Library/IUserManager.cs (+7 -3)
📝 MediaBrowser.Model/Cryptography/ICryptoProvider.cs (+0 -4)

📄 Description

Also removes 2 unused methods.


🔄 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/1926 **Author:** [@Bond-009](https://github.com/Bond-009) **Created:** 10/20/2019 **Status:** ✅ Merged **Merged:** 11/22/2019 **Merged by:** [@dkanada](https://github.com/dkanada) **Base:** `master` ← **Head:** `auth` --- ### 📝 Commits (4) - [`fef35d0`](https://github.com/jellyfin/jellyfin/commit/fef35d0505c6485954b1eb233a5ea67e33fe1138) Add clearer exceptions, warnings and docs - [`24c076e`](https://github.com/jellyfin/jellyfin/commit/24c076eff300726f55ee856a230c0c6a25c462de) Update Emby.Server.Implementations/Library/UserViewManager.cs - [`2b1e766`](https://github.com/jellyfin/jellyfin/commit/2b1e766c17031536516ec081c3a5cabde82cb580) Update MediaBrowser.Controller/Library/IUserManager.cs - [`d7335f6`](https://github.com/jellyfin/jellyfin/commit/d7335f6ae6c9cb2b5000ca9be9c8b7f6f268af2a) Reword exception message ### 📊 Changes **10 files changed** (+63 additions, -57 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/Cryptography/CryptographyProvider.cs` (+3 -7) 📝 `Emby.Server.Implementations/Library/DefaultAuthenticationProvider.cs` (+4 -1) 📝 `Emby.Server.Implementations/Library/UserManager.cs` (+6 -14) 📝 `Emby.Server.Implementations/Library/UserViewManager.cs` (+10 -10) 📝 `Emby.Server.Implementations/Session/SessionManager.cs` (+14 -13) 📝 `MediaBrowser.Api/UserLibrary/UserViewsService.cs` (+7 -1) 📝 `MediaBrowser.Common/Cryptography/PasswordHash.cs` (+3 -3) 📝 `MediaBrowser.Controller/Entities/Folder.cs` (+9 -1) 📝 `MediaBrowser.Controller/Library/IUserManager.cs` (+7 -3) 📝 `MediaBrowser.Model/Cryptography/ICryptoProvider.cs` (+0 -4) </details> ### 📄 Description Also removes 2 unused methods. --- <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 05:53:22 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#8951