bitwarden/browser
1
0
Fork 0
mirror of https://github.com/bitwarden/browser synced 2026-02-10 21:50:15 +00:00
Code Issues Releases Wiki Activity

Fix min and default values in kdf migration

Browse Source
This commit is contained in:
Bernd Schoolmann
2025-11-03 15:15:38 +01:00
parent 60be9a2e1f
commit a456e7e330
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
libs/common/src/key-management/encrypted-migrator/migrations/minimum-kdf-migration.spec.ts
View File

@@ -140,7 +140,7 @@ describe("MinimumKdfMigration", () => {
// Verify the PBKDF2KdfConfig has the correct iteration count
const kdfConfigArg = (mockChangeKdfService.updateUserKdfParams as jest.Mock).mock.calls[0][1];
expect(kdfConfigArg.iterations).toBe(PBKDF2KdfConfig.ITERATIONS.min);
expect(kdfConfigArg.iterations).toBe(PBKDF2KdfConfig.ITERATIONS.defaultValue);
});
it("should throw error when userId is null", async () => {

2
libs/common/src/key-management/encrypted-migrator/migrations/minimum-kdf-migration.ts
View File

@@ -54,7 +54,7 @@ export class MinimumKdfMigration implements EncryptedMigration {
const kdfConfig = await this.kdfConfigService.getKdfConfig(userId);
if (
kdfConfig.kdfType !== KdfType.PBKDF2_SHA256 ||
kdfConfig.iterations >= PBKDF2KdfConfig.ITERATIONS.defaultValue
kdfConfig.iterations >= PBKDF2KdfConfig.ITERATIONS.min
) {
return "noMigrationNeeded";
}