Skip to content

feat(validation): [PM-32626]#7064

Draft
Patrick-Pimentel-Bitwarden wants to merge 1 commit intomainfrom
auth/pm-32626/standardize-unlock-authentication-validation
Draft

feat(validation): [PM-32626]#7064
Patrick-Pimentel-Bitwarden wants to merge 1 commit intomainfrom
auth/pm-32626/standardize-unlock-authentication-validation

Conversation

@Patrick-Pimentel-Bitwarden
Copy link
Contributor

@Patrick-Pimentel-Bitwarden Patrick-Pimentel-Bitwarden commented Feb 24, 2026

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-32626

📔 Objective

  • Standardized KDF/salt validation: Introduced KdfSettingsValidator.ValidateAuthenticationAndUnlockData() (src/Core/Utilities/KdfSettingsValidator.cs) to consolidate validation that authentication and unlock data share consistent KDF settings and salt values. This is now used by RegisterFinishRequestModel.

📸 Screenshots

…lidation - Initial pass at standardizing the validation for the RegisterFinishRequestModel.
@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2026

Logo
Checkmarx One – Scan Summary & Detailsab0d8198-8f4e-4270-b49f-a294a1f1e3b2

Great job! No new security vulnerabilities introduced in this pull request

@Patrick-Pimentel-Bitwarden Patrick-Pimentel-Bitwarden changed the title feat(validation): [PM-32626] Standardize Unlock and Authentication Va… feat(validation): [PM-32626] Feb 25, 2026
@Patrick-Pimentel-Bitwarden Patrick-Pimentel-Bitwarden force-pushed the auth/pm-32626/standardize-unlock-authentication-validation branch from 54b3af6 to 303ac1c Compare February 25, 2026 13:47
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant