-
Notifications
You must be signed in to change notification settings - Fork 91
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
attestation-agent/Attesters: refactor the trait of Attester #284
attestation-agent/Attesters: refactor the trait of Attester #284
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a small remark about the signature, but I agree it makes a lot of sense to move the encoding to the caller site.
b176633
to
a5a3724
Compare
a5a3724
to
1aadb3e
Compare
Change the API of `get_evidence` function to just performing getting evidence via calling the underlying hardware drivers. Fixes: confidential-containers#283 Signed-off-by: Xynnn007 <xynnn@linux.alibaba.com>
1aadb3e
to
705bff3
Compare
Rebased upstream to resolve conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Hopefully this won't cause any mismatch with the validation in the AS. I don't think it will.
Change the API of
get_evidence
function to just performing getting evidence via calling the underlying hardware drivers.Fixes: #283
Please do not merge this until the v0.8.0 cycle.
cc @jialez0 @fitzthum @mkulke please take a look if these changes are good for the architecture/concrete attester implementation.