Disambiguate JSON deserialization of AuthenticatorTransport

This broke in Jackson 2.12.0-rc1, seems like the deserializer somehow started
choosing the `valueOf` function instead of the `of` function. At first this
seemed like a regression in Jackson, but it was ambiguous on our side either
way.

Author: emlun

NEWS

@@ -6,6 +6,7 @@ Changes:
  ** Unless `RelyingParty.allowUntrustedAttestation` is set to `false`, unknown
     attestation statements will now pass as untrusted attestations, instead of
     throwing an IllegalArgumentException.
+* Disambiguated Jackson deserialization of class `AuthenticatorTransport`
 
 New features:
 

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

@@ -25,6 +25,7 @@
 package com.yubico.webauthn.data;
 
 
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
 import com.yubico.internal.util.json.JsonStringSerializable;
 import com.yubico.internal.util.json.JsonStringSerializer;
@@ -92,6 +93,7 @@ public static AuthenticatorTransport[] values() {
      * #INTERNAL}, returns that constant instance. Otherwise returns a new instance containing <code>id</code>.
      * @see #valueOf(String)
      */
+    @JsonCreator
     public static AuthenticatorTransport of(@NonNull String id) {
         return Stream.of(values())
             .filter(v -> v.getId().equals(id))
@@ -114,7 +116,7 @@ public static AuthenticatorTransport valueOf(String name) {
             case "BLE": return BLE;
             case "INTERNAL": return INTERNAL;
         default:
-            throw new IllegalArgumentException("No enum constant com.yubico.webauthn.data.AuthenticatorTransport." + name);
+            throw new IllegalArgumentException("No constant com.yubico.webauthn.data.AuthenticatorTransport." + name);
         }
     }