Fix Login With Device without MasterPassword #1473

Closed
opened 2026-04-06 01:55:58 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @BlackDex on 9/1/2023

It looks like either something changed in the latest v2023.8.x versions, or when using Biometrics to login, but the MasterPasswordHash is an optional value during the Approve action.

This PR makes the MasterPasswordHash an optional value which resolves this issues. Bitwarden works the same way.

I also changed the EncKey to an Option in the database since empty strings as a default value is not nice in databases, better to use null in these cases.

Fixes #3819

*Originally created by @BlackDex on 9/1/2023* It looks like either something changed in the latest v2023.8.x versions, or when using Biometrics to login, but the MasterPasswordHash is an optional value during the Approve action. This PR makes the MasterPasswordHash an optional value which resolves this issues. Bitwarden works the same way. I also changed the EncKey to an Option in the database since empty strings as a default value is not nice in databases, better to use `null` in these cases. Fixes #3819
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/vaultwarden#1473