Skip to content

Commit 8136c91

Browse files
committed
Make user verification required by default
1 parent 0951892 commit 8136c91

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/Identity/Core/src/IdentityPasskeyOptions.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,15 +69,16 @@ public class IdentityPasskeyOptions
6969
/// </para>
7070
/// <para>
7171
/// Possible values are "required", "preferred", and "discouraged".
72+
/// If set to <see langword="null"/>, the effective value is "preferred".
7273
/// </para>
7374
/// <para>
74-
/// If left <see langword="null"/>, the browser defaults to "preferred".
75+
/// The default value is "required".
7576
/// </para>
7677
/// <para>
7778
/// See <see href="https://www.w3.org/TR/webauthn-3/#enumdef-userverificationrequirement"/>.
7879
/// </para>
7980
/// </remarks>
80-
public string? UserVerificationRequirement { get; set; }
81+
public string? UserVerificationRequirement { get; set; } = "required";
8182

8283
/// <summary>
8384
/// Gets or sets the extent to which the server desires to create a client-side discoverable credential.

src/Identity/test/Identity.Test/IdentityPasskeyOptionsTest.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,9 @@ public void VerifyDefaultOptions()
1212

1313
Assert.Equal(TimeSpan.FromMinutes(5), options.AuthenticatorTimeout);
1414
Assert.Equal(32, options.ChallengeSize);
15+
Assert.Equal("preferred", options.ResidentKeyRequirement);
16+
Assert.Equal("required", options.UserVerificationRequirement);
1517
Assert.Null(options.ServerDomain);
16-
Assert.Null(options.UserVerificationRequirement);
17-
Assert.Null(options.ResidentKeyRequirement);
1818
Assert.Null(options.AttestationConveyancePreference);
1919
Assert.Null(options.AuthenticatorAttachment);
2020
Assert.Null(options.IsAllowedAlgorithm);

0 commit comments

Comments
 (0)