fido-alliance / iot-fdo-conformance-tools

FIDO Alliances Reference FDO Implementation and Conformance Testing
https://fidoalliance.org/intro-to-fido-device-onboard/
Other
5 stars 2 forks source link

[BUG] Invalid Value sent in Protected header in message 61 #30

Closed Sai-Anudeep47 closed 1 year ago

Sai-Anudeep47 commented 1 year ago

To simplify issue resolution process, please provide network logs, and or test voucher. to2_msg61_invalid_protected_header_logs.txt

What part of the spec are you testing?

What protocol are having issue with?

Issue description

Log is attached for reference

yackermann commented 1 year ago

So the spec is kind of confusing on that part:

Here is a definitions in the protected header:

https://drafts.fidoalliance.org/internet-of-things-specs/stable-links-to-latest/FIDO-IoT-spec.html#cose-signatures

$$COSEProtectedHeaders /= ()

The only time the header contains any algorithms is in Voucher:

$COSEProtectedHeaders //= (
    1: OVSignType
)

I do agree that we should be specifying it, and have added it to the latest build.

But it does need to be addressed in the spec: @GeofCooper