You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I found a couple of issues causing this to fail when testing the passwordless feature of the demo.
Registration is failing out of the box because when posting to /makeCredential extensions.credOpts used to be true and is now {rk: true}, This is causing a binding error on the AuthenticatorAttestationRawResponse object.
/makeCredential was further failing because when storing the credential to DemoStorage Descriptor is null because success.Result.CredentialId is no longer being populated in AuthenticatorAttestationResponse.cs. This fails silently on registration and appears to be successful, but Sign In will fail because the credential can't be found when doing /assertionOptions.
The text was updated successfully, but these errors were encountered:
Thanks for testing these issues. We have a very long running branch #277 that we need to merge. @aseigler Do you even think it's possible to merge that branch or should we just create new smaller PR's that add some of the changes you did?
I found a couple of issues causing this to fail when testing the passwordless feature of the demo.
Registration is failing out of the box because when posting to /makeCredential extensions.credOpts used to be true and is now {rk: true}, This is causing a binding error on the AuthenticatorAttestationRawResponse object.
/makeCredential was further failing because when storing the credential to DemoStorage Descriptor is null because success.Result.CredentialId is no longer being populated in AuthenticatorAttestationResponse.cs. This fails silently on registration and appears to be successful, but Sign In will fail because the credential can't be found when doing /assertionOptions.
The text was updated successfully, but these errors were encountered: