Added credProps.authenticatorDisplayName

[fdennis]

No description provided.

[github-actions]

Test Results

2 218 tests  +12   2 210 ✅ +12   1m 31s ⏱️ +33s
   45 suites ± 0       8 💤 ± 0 
   45 files   ± 0       0 ❌ ± 0 

Results for commit a1a3e2f. ± Comparison against base commit 563e5c3.

♻️ This comment has been updated with latest results.

[warchildmd]

When will this be merged? 1Password is sending this field and it's failing because of FAIL_ON_UNKNOWN_PROPERTIES

[JDussaussois]

@warchildmd : Same here, i'm interested about a merge 👍

[emlun]

@warchildmd Which version of webauthn-server-core are you using? That issue should have been fixed in release 2.5.2.

[JDussaussois]

@emlun : I'm actually using release 2.5.2 and the error is still here :
UnrecognizedPropertyException: Unrecognized field "authenticatorDisplayName"

[emlun]

Ok, we can fast track a bug fix for that then. Could you please open a new issue and include the full stack trace of the exception?

[JDussaussois]

My bad, after a deep check, i'm actually using 2.5.1 in my deployed env, i'll update to 2.5.2 and check, thanks for the feedback

[emlun]

@JDussaussois Did the upgrade fix your problem?

[emlun]

Looking good! I tweaked the JavaDocs a bit, and while doing that I realized that the new version of the extension is also applicable to assertions, so we need to parse and expose credProps in ClientAssertionExtensionOutputs and AssertionResult too. I'll push those changes on top of this, they're fully analogous to what's here already.