Merge pull request #222 from Yubico/aaron/32-byte-protocol-2

fix: Remove incorrect auth token truncation from various commands

Author: DennisDyallo

Yubico.YubiKey/src/Yubico/YubiKey/Fido2/Commands/BioEnrollmentCommand.cs

@@ -188,7 +188,7 @@ public BioEnrollmentCommand(
             // The pinUvAuthToken is an encrypted value, so there's no need to
             // overwrite the array.
             byte[] authParam = authProtocol.AuthenticateUsingPinToken(pinUvAuthToken.ToArray(), message);
-            PinUvAuthParam = new ReadOnlyMemory<byte>(authParam, 0, 16);
+            PinUvAuthParam = new ReadOnlyMemory<byte>(authParam);
             PinUvAuthProtocol = authProtocol.Protocol;
         }
 

Yubico.YubiKey/src/Yubico/YubiKey/Fido2/Commands/ConfigCommand.cs

@@ -180,7 +180,7 @@ public ConfigCommand(
             // The pinUvAuthToken is an encrypted value, so there's no need to
             // overwrite the array.
             byte[] authParam = authProtocol.AuthenticateUsingPinToken(pinUvAuthToken.ToArray(), message);
-            PinUvAuthParam = new ReadOnlyMemory<byte>(authParam, 0, 16);
+            PinUvAuthParam = new ReadOnlyMemory<byte>(authParam);
             PinUvAuthProtocol = authProtocol.Protocol;
         }
 

Yubico.YubiKey/src/Yubico/YubiKey/Fido2/Commands/CredentialManagementCommand.cs

@@ -184,7 +184,7 @@ public CredentialManagementCommand(
             // The pinUvAuthToken is an encrypted value, so there's no need to
             // overwrite the array.
             byte[] authParam = authProtocol.AuthenticateUsingPinToken(pinUvAuthToken.ToArray(), message);
-            PinUvAuthParam = new ReadOnlyMemory<byte>(authParam, 0, 16);
+            PinUvAuthParam = new ReadOnlyMemory<byte>(authParam);
             PinUvAuthProtocol = authProtocol.Protocol;
         }