trussed-dev / fido2-tests

Test suite for FIDO2, U2F, and other security key functions
Other
48 stars 38 forks source link

Fix test_allow_list_id_wrong_type and test_allow_list_missing_id #15

Closed andrewkozlik closed 4 years ago

andrewkozlik commented 4 years ago

In TestGetAssertion::test_allow_list_id_wrong_type and TestGetAssertion::test_allow_list_missing_id the "type" attribute should be "public-key" instead of b"public-key", i.e. str instead of bytes. Currently the tests succeed even for an authenticator that doesn't check the presence and data type of the "id" attribute, because the authenticator will return an error caused by the wrong data type of the "type" attribute.