FeatureFuel
2

Improve 2FA Token Missing Logging

Source: dani-garcia/vaultwarden#4613 · opened by @t0rb3n
We see a lot of
[error][ERROR] 2FA token not provided in our logs without being able to map this to a specific user.

My suggestion is to improve logging in src/api/idenitity.rs
by appending the same output as password login failed, e.g. format!("IP: {}. Username: {}.", ip.ip, username) (Copied from

So that line could become something like this:
None => err_json!(_json_err_twofactor(&twofactor_ids, &user.uuid, conn).await?, "2FA token not provided ", format!("IP: {}. Username: {}.", ip.ip, user.username)), (Not sure what properties in user exists)

No pledges yet. Be the first to back this.

Make a pledge

Pledge your monetary support if this feature is added.

$

Comments

No comments yet.

Replying to

Add a comment

What do you think about this feature request?


Similar requests