Make Jackson option PROPAGATE_TRANSIENT_MARKER not needed

Author: emlun

NEWS

@@ -14,6 +14,11 @@ New features:
 * Updated README and JavaDoc to use the "passkey" term and provide more guidance
   around passkey use cases.
 
+Fixes:
+
+* Made Jackson setting `PROPAGATE_TRANSIENT_MARKER` unnecessary for JSON
+  serialization with Jackson version 2.15.0-rc1 and later.
+
 
 == Version 2.4.1 ==
 

webauthn-server-core/src/main/java/com/yubico/webauthn/data/AuthenticatorAttestationResponse.java

@@ -82,7 +82,10 @@ public class AuthenticatorAttestationResponse implements AuthenticatorResponse {
   private final SortedSet<AuthenticatorTransport> transports;
 
   /** The {@link #attestationObject} parsed as a domain object. */
-  @NonNull @JsonIgnore private final transient AttestationObject attestation;
+  @NonNull
+  @JsonIgnore
+  @Getter(onMethod = @__({@JsonIgnore}))
+  private final transient AttestationObject attestation;
 
   @NonNull
   @JsonIgnore

yubico-util/src/main/java/com/yubico/internal/util/JacksonCodecs.java

@@ -3,7 +3,6 @@
 import com.fasterxml.jackson.annotation.JsonInclude.Include;
 import com.fasterxml.jackson.core.Base64Variants;
 import com.fasterxml.jackson.databind.DeserializationFeature;
-import com.fasterxml.jackson.databind.MapperFeature;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.json.JsonMapper;
 import com.fasterxml.jackson.databind.node.ObjectNode;
@@ -22,7 +21,6 @@ public static ObjectMapper cbor() {
   public static ObjectMapper json() {
     return JsonMapper.builder()
         .configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, true)
-        .configure(MapperFeature.PROPAGATE_TRANSIENT_MARKER, true)
         .serializationInclusion(Include.NON_ABSENT)
         .defaultBase64Variant(Base64Variants.MODIFIED_FOR_URL)
         .addModule(new Jdk8Module())