Add more de-/serialization tests #54
Assignees
Comments
robin-nitrokey
added a commit
that referenced
this issue
Jun 24, 2024
This patch adds de-/serialization tests for the modules ctap1, ctap2::client_pin and ctap2::large_blobs. See also: #54
robin-nitrokey
added a commit
that referenced
this issue
Jun 24, 2024
This patch adds de-/serialization tests for the modules ctap1, ctap2::client_pin and ctap2::large_blobs. See also: #54
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It is easy to break serialization, for example because enums are serialized using their variant index by default, byte slices are serialized as sequences of integers instead of hex strings, and reordering or removing fields can break
serde-indexed. Therefore we should have basic de-/serialization tests for all requests and responses.ctap1ctap2ctap2::client_pinctap2::credential_managementctap2::get_assertionctap2::get_infoctap2::large_blobsctap2::make_credentialwebauthnThe text was updated successfully, but these errors were encountered: