Skip to content

Commit 55ed29d

Browse files
committed
Clean up should_use_uv
1 parent dd2a178 commit 55ed29d

1 file changed

Lines changed: 3 additions & 16 deletions

File tree

crates/yubikit/src/webauthn/client.rs

Lines changed: 3 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1144,7 +1144,7 @@ impl<C: Connection + 'static, U: UserInteraction, D: ClientDataCollector> WebAut
11441144
}
11451145

11461146
Err(ClientError::ConfigurationUnsupported(
1147-
"user verification not configured".into(),
1147+
"user verification required but not configured/supported".into(),
11481148
))
11491149
}
11501150

@@ -1166,21 +1166,8 @@ impl<C: Connection + 'static, U: UserInteraction, D: ClientDataCollector> WebAut
11661166
|| info.options.contains_key("bioEnroll");
11671167

11681168
match uv_requirement {
1169-
UserVerificationRequirement::Required => {
1170-
if !uv_configured {
1171-
return Err(ClientError::ConfigurationUnsupported(
1172-
"user verification required but not configured".into(),
1173-
));
1174-
}
1175-
Ok(true)
1176-
}
1177-
UserVerificationRequirement::Preferred => {
1178-
if uv_supported && uv_configured {
1179-
Ok(true)
1180-
} else {
1181-
Ok(false)
1182-
}
1183-
}
1169+
UserVerificationRequirement::Required => Ok(true),
1170+
UserVerificationRequirement::Preferred => Ok(uv_supported),
11841171
UserVerificationRequirement::Discouraged => {
11851172
// Even if discouraged, UV may still be needed:
11861173
// - alwaysUv option

0 commit comments

Comments
 (0)