fix: store mfa default values when missing (#64)

teamhanko · May 7, 2024 · 0b18ba8 · 0b18ba8
1 parent c0af643
commit 0b18ba8
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/server/api/middleware/webauthn.go b/server/api/middleware/webauthn.go
@@ -44,16 +44,14 @@ func WebauthnMiddleware(persister persistence.Persister) echo.MiddlewareFunc {
 }
 
 func setWebauthnClientCtx(ctx echo.Context, cfg models.Config, persister persistence.Persister) error {
-	var passkeyConfig models.WebauthnConfig
-
 	err := createPasskeyClient(ctx, cfg.WebauthnConfig)
 	if err != nil {
 		ctx.Logger().Error(err)
 		return err
 	}
 
-	if cfg.MfaConfig == nil {
-		cfg.MfaConfig, err = createDefaultMfaConfig(persister, passkeyConfig)
+	if cfg.MfaConfig == nil || cfg.MfaConfig.ID == uuid.Nil {
+		cfg.MfaConfig, err = createDefaultMfaConfig(persister, cfg.WebauthnConfig)
 		if err != nil {
 			ctx.Logger().Error(err)
 			return err