Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snp add report claim #659

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

fitzthum
Copy link
Member

The EAR broker expects the verifier to include report_data in the TCB claims as a signal that the verifier has checked the binding of the report data and the hw evidence.

Since we do check report_data in the SNP verifier, add the claim. Also add init_data claim for similar reasons.

EAR tokens expect to find a report_data field in the TCB Claims as a
signal that the verifier has checked the binding of the report data and
the evidence.

The SNP verifier does check the report data field, but it does not
report it.

This should not affect the az-snp verifier which will insert its own
report_data on top of this field.

Signed-off-by: Tobin Feldman-Fitzthum <[email protected]>
Since the SNP verifier also checks the init data, include the init_data
field in the tcb claims.

This will allow EAR tokens to contain the init_data_claims.

Signed-off-by: Tobin Feldman-Fitzthum <[email protected]>
@fitzthum fitzthum requested a review from a team as a code owner January 10, 2025 17:10
@fitzthum
Copy link
Member Author

Need this for the release btw @Xynnn007 @mkulke

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant