Skip to content

Commit 184f70f

Browse files
authored
Resolve StoredCredential.Descriptor at access(?) (#528)
* Just resolve the descriptor at runtime * Missing ; * Update StoredCredential.cs
1 parent df37692 commit 184f70f

File tree

4 files changed

+6
-5
lines changed

4 files changed

+6
-5
lines changed

BlazorWasmDemo/Server/Controllers/UserController.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,6 @@ public async Task<string> CreateCredentialAsync([FromRoute] string username, [Fr
162162
{
163163
AttestationFormat = result.Result.AttestationFormat,
164164
Id = result.Result.Id,
165-
Descriptor = new PublicKeyCredentialDescriptor(result.Result.Id),
166165
PublicKey = result.Result.PublicKey,
167166
UserHandle = result.Result.User.Id,
168167
SignCount = result.Result.SignCount,

Demo/Controller.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ public async Task<JsonResult> MakeCredential([FromBody] AuthenticatorAttestation
112112
DemoStorage.AddCredentialToUser(options.User, new StoredCredential
113113
{
114114
Id = success.Result.Id,
115-
Descriptor = new PublicKeyCredentialDescriptor(success.Result.Id),
116115
PublicKey = success.Result.PublicKey,
117116
UserHandle = success.Result.User.Id,
118117
SignCount = success.Result.SignCount,

Demo/TestController.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public async Task<JsonResult> MakeCredentialResultTestAsync([FromBody] Authentic
9696
// 3. Store the credentials in db
9797
_demoStorage.AddCredentialToUser(options.User, new StoredCredential
9898
{
99-
Descriptor = new PublicKeyCredentialDescriptor(success.Result.Id),
99+
Id = success.Result.Id,
100100
PublicKey = success.Result.PublicKey,
101101
UserHandle = success.Result.User.Id,
102102
SignCount = success.Result.SignCount

Src/Fido2.Development/StoredCredential.cs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ public class StoredCredential
99
/// <summary>
1010
/// The Credential ID of the public key credential source.
1111
/// </summary>
12-
public byte[] Id { get; set; }
12+
public required byte[] Id { get; set; }
1313

1414
/// <summary>
1515
/// The credential public key of the public key credential source.
@@ -52,7 +52,10 @@ public class StoredCredential
5252

5353
public byte[] UserId { get; set; }
5454

55-
public PublicKeyCredentialDescriptor Descriptor { get; set; }
55+
/// <summary>
56+
/// Exposes an Descriptor Object for this credential, used as input to the library for certain operations.
57+
/// </summary>
58+
public PublicKeyCredentialDescriptor Descriptor => new(PublicKeyCredentialType.PublicKey, Id, Transports);
5659

5760
public byte[] UserHandle { get; set; }
5861

0 commit comments

Comments
 (0)