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

Refactor remote attestation code #198

Merged
merged 15 commits into from Dec 30, 2019
Merged

Refactor remote attestation code #198

merged 15 commits into from Dec 30, 2019

Conversation

@mssun
Copy link
Member

mssun commented Dec 30, 2019

Description

Introduce teaclave_attestation, a new create for attestation related code.

This PR refactor RA related code and move them into the teaclave_attestation crate.

For the teaclave_attestation, we start to try thiserror and anyhow for error management.

Type of change (select applied and DELETE the others)

  • Code polishing

How Has This Been Tested?

CI: https://ci.mesalock-linux.org/mssun/incubator-mesatee/275

Checklist (check ALL before submitting PR, even not applicable)

  • Fork the repo and create your branch from master.
  • If you've added code that should be tested, add tests.
  • If you've changed APIs, update the documentation.
  • Ensure the tests pass (see CI results).
  • Make sure your code lints/format.
mssun added 10 commits Dec 29, 2019
mssun added 4 commits Dec 30, 2019
@mssun

This comment has been minimized.

Copy link
Member Author

mssun commented Dec 30, 2019

I also add a verifier.rs file (https://github.com/apache/incubator-teaclave/pull/198/files#diff-dd6dee8420bdab6e40cf4171574f1aec) for TLS verifier for RA. Basically, I create a new struct called SgxQuoteVerifier to wrap around EnclaveAttr and rewrite the original EnclaveAttr struct.

@mssun mssun merged commit 21cf3bb into apache:master Dec 30, 2019
@mssun mssun deleted the mssun:ra branch Dec 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.