Closed jbb01 closed 1 year ago
Thanks for the report! For most normal use of the library this was fixed by commit 5d1effca2b3ce9312ec23c4e8b4b2acbfdad25d1 in release 2.4.1
(see also the Jackson issue referenced in the diff), but I realize now that this fix only applies to the ObjectMapper
used internally by the library. I suppose the @JsonIgnore
annotation is necessary after all. This will be fixed in the next release.
Thanks for your fast response. The commit you linked helped in fixing my problem.
Fixed and will be released in version 2.5.0
. Thanks again for the report!
When serializing an
AuthenticatorAttestationResponse
with Jackson, an additional propertyattestation
is present. This causes anUnrecognizedPropertyException
when trying to deserialize that json back into anAuthenticatorAttestationResponse
. The problem seems to be caused by a missing@JsonIgnore
on the getterAuthenticatorAttestationResponse::getAttestation()
.The following code
first prints
and then throws
This has been tested with
com.yubico:webauthn-server-core:2.4.1
,com.fasterxml.jackson.core:jackson-core:2.15.0
and Temurin-17.0.6+10. TheAuthenticatorAttestationResponse
object was created by the following code