Skip to content

[P0-1][CRITICAL] Replace MD5 password hashing with PBKDF2 #3

@cct08311github

Description

@cct08311github

Login uses bare Utils.GetMD5String(password) - unsalted MD5. Vulnerable to rainbow table attacks. Fails CISSP/SOC2 compliance.

Fix

  • Add PasswordHashHelper with PBKDF2 (ASP.NET Core Identity PasswordHasher)
  • Transparent verify-and-upgrade migration on login
  • Apply across all demo projects: AccountController, DataContext, FrameworkUserVM, ChangePasswordVM, FrameworkUserImportVM, FrameworkTenantVM
  • Mark Utils.GetMD5String [Obsolete]
  • Fix GetMD5Stream memory bomb
  • DB migration: ALTER TABLE FrameworkUser ALTER COLUMN Password NVARCHAR(256)

Risk: Medium | Effort: 2-3 days

Metadata

Metadata

Assignees

No one assigned

    Labels

    P0Critical prioritysecuritySecurity related

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions