mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-07-16 05:43:54 +03:00
[PR #6818] [MERGED] Replace PBKDF2-SHA1 with PBKDF2-SHA512 #11142
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/6818
Author: @Bond-009
Created: 11/11/2021
Status: ✅ Merged
Merged: 11/14/2021
Merged by: @crobibero
Base:
master← Head:password📝 Commits (1)
5265b3eReplace PBKDF2-SHA1 with PBKDF2-SHA512📊 Changes
9 files changed (+88 additions, -113 deletions)
View changed files
📝
Emby.Server.Implementations/Cryptography/CryptographyProvider.cs(+55 -35)📝
Emby.Server.Implementations/LiveTv/Listings/SchedulesDirect.cs(+2 -1)📝
Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs(+11 -24)📝
Jellyfin.Server.Implementations/Users/UserManager.cs(+1 -7)➖
MediaBrowser.Common/Cryptography/CryptoExtensions.cs(+0 -35)📝
MediaBrowser.Model/Cryptography/Constants.cs(+8 -3)📝
MediaBrowser.Model/Cryptography/ICryptoProvider.cs(+8 -5)📝
MediaBrowser.Model/Cryptography/PasswordHash.cs(+1 -1)📝
tests/Jellyfin.Model.Tests/Cryptography/PasswordHashTests.cs(+2 -2)📄 Description
This also migrates already created passwords on login
Source for the number of iterations:
https://cheatsheetseries.owasp.org/cheatsheets/Password_Storage_Cheat_Sheet.html#pbkdf2
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.