[PR #3423] [MERGED] Remove EasyPassword from Authentication providers #9336

Closed
opened 2025-12-22 07:25:26 +01:00 by backuprepo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/3423
Author: @crobibero
Created: 6/23/2020
Status: Merged
Merged: 7/4/2020
Merged by: @joshuaboniface

Base: masterHead: easypassword


📝 Commits (5)

  • 68e1eca Remove EasyPassword from Authentication providers
  • 3d87c4c Fix EasyPassword setting
  • f4d8e0e Merge remote-tracking branch 'upstream/master' into easypassword
  • 3169b0a fix merge
  • 44a8ea6 implement ChangeEasyPassword from legacy provider

📊 Changes

4 files changed (+11 additions, -40 deletions)

View changed files

📝 Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs (+0 -25)
📝 Jellyfin.Server.Implementations/Users/InvalidAuthProvider.cs (+0 -12)
📝 Jellyfin.Server.Implementations/Users/UserManager.cs (+11 -1)
📝 MediaBrowser.Controller/Authentication/IAuthenticationProvider.cs (+0 -2)

📄 Description

This doesn't change any functionality as EasyPassword was already stored and retrieved from the db.

CI failing due to:

##[warning]Error MethodMustNotBeChanged: Publicly-accessible method 'MediaBrowser.Controller.Authentication.IAuthenticationProvider.ChangeEasyPassword' was changed or removed.
##[warning]Error MethodMustNotBeChanged: Publicly-accessible method 'MediaBrowser.Controller.Authentication.IAuthenticationProvider.GetEasyPasswordHash' was changed or removed.

🔄 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/3423 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 6/23/2020 **Status:** ✅ Merged **Merged:** 7/4/2020 **Merged by:** [@joshuaboniface](https://github.com/joshuaboniface) **Base:** `master` ← **Head:** `easypassword` --- ### 📝 Commits (5) - [`68e1eca`](https://github.com/jellyfin/jellyfin/commit/68e1ecaaf9626c9596c833571bb591a326449dc8) Remove EasyPassword from Authentication providers - [`3d87c4c`](https://github.com/jellyfin/jellyfin/commit/3d87c4c1b6bca920f444b4a16a99553d0ff6879e) Fix EasyPassword setting - [`f4d8e0e`](https://github.com/jellyfin/jellyfin/commit/f4d8e0e20cca81b68919ac8d242fa793612d8c85) Merge remote-tracking branch 'upstream/master' into easypassword - [`3169b0a`](https://github.com/jellyfin/jellyfin/commit/3169b0af8917ac0bfd189fdb2f261be774b8b869) fix merge - [`44a8ea6`](https://github.com/jellyfin/jellyfin/commit/44a8ea6bee3cf3fefc3d290dbaaa8c8e1554868f) implement ChangeEasyPassword from legacy provider ### 📊 Changes **4 files changed** (+11 additions, -40 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs` (+0 -25) 📝 `Jellyfin.Server.Implementations/Users/InvalidAuthProvider.cs` (+0 -12) 📝 `Jellyfin.Server.Implementations/Users/UserManager.cs` (+11 -1) 📝 `MediaBrowser.Controller/Authentication/IAuthenticationProvider.cs` (+0 -2) </details> ### 📄 Description This doesn't change any functionality as EasyPassword was already stored and retrieved from the db. CI failing due to: ``` ##[warning]Error MethodMustNotBeChanged: Publicly-accessible method 'MediaBrowser.Controller.Authentication.IAuthenticationProvider.ChangeEasyPassword' was changed or removed. ##[warning]Error MethodMustNotBeChanged: Publicly-accessible method 'MediaBrowser.Controller.Authentication.IAuthenticationProvider.GetEasyPasswordHash' was changed or removed. ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
backuprepo 2025-12-22 07:25:26 +01:00
Sign in to join this conversation.
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#9336
No description provided.