connectedhomeip icon indicating copy to clipboard operation
connectedhomeip copied to clipboard

Add Device Attestation validation to `chip-cert` tool for YAML test support

Open tcarmelveilleux opened this issue 4 years ago • 1 comments

Problem

There is no "freestanding" implementation of the Device Attestation validation procedure that could be executed by implementers and by CSG test harness based on responses obtained, to ascertain the current validity of a given implementation.

Proposed Solution

In order to automate Device Attestation Procedure testing, we could have a command-line tool that can validate the AttestationResponse and and provide a result and logs, so that it can be wrapped in a delegated YAML command.

This will allow executing an instrumented view of the device attestation procedure.

The tooling could then be replaced in the future with an equivalent independent implementation.

Two commands are necessary:

  • Validation of AttestationSignature on AttestationResponse
  • Validation of AttestationSignature on NOCSRResponse

The common arguments for both:

  • PAI cert in DER, encoded as hex
  • DAC cert in DER, encoded as hex
  • Attestation Challenge from secure session, encoded as hex
  • Nonce (AttestationNonce, CSRNonce), encoded as hex
  • AttestationSignature (from either AttestationResponse or NOCSRResponse), encoded as hex

The specific arguments are either:

  • AttestationElements from AttestationResponse
  • NOCSRElements from NOCSRResponse

tcarmelveilleux avatar Nov 17 '21 15:11 tcarmelveilleux

@tcarmelveilleux This seems important - should it remain in the 1.0 list?

holbrookt avatar Jan 31 '22 23:01 holbrookt

Impact: 8 Device Attestation TCs Priority: High

naveenkommareddi avatar Nov 02 '22 02:11 naveenkommareddi

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar May 02 '23 00:05 stale[bot]

This stale issue has been automatically closed. Thank you for your contributions.

stale[bot] avatar May 18 '23 18:05 stale[bot]

This is done.

tcarmelveilleux avatar Nov 27 '23 02:11 tcarmelveilleux