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

Add TrustedIdentitiesVerifier #124

Merged
merged 1 commit into from
Jun 30, 2023
Merged

Conversation

nick-mobilecoin
Copy link
Collaborator

@nick-mobilecoin nick-mobilecoin commented Jun 30, 2023

The TrustedIdentitiesVerifier will verify that one of the provided
identities matches the enclave's identity.

The `TrustedIdentitiesVerifier` will verify that one of the provided
identities matches the enclave's identity.
@nick-mobilecoin
Copy link
Collaborator Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@github-actions github-actions bot added the size/L Large PRs label Jun 30, 2023
@meowblecoinbot meowblecoinbot requested a review from a team June 30, 2023 15:22
@github-actions
Copy link

github-actions bot commented Jun 30, 2023

❌ Unreviewed dependencies found

Crate Version Reviews (N/2) LoC Left-Pad Index Geiger Flags
yare 1.0.2 0 89 11 0 ____
mc-sgx-dcap-sys-types 0.7.0 0 124 3 0 CB__
assert_matches 1.5.0 0 209 84 0 ____
mc-sgx-core-sys-types 0.7.0 0 226 5 0 CB__
subtle 2.5.0 0 440 147 9 ____
hex 0.4.3 0 474 145 0 ____
displaydoc 0.2.4 0 600 59 20 ____
x509-cert 0.2.3 0 1963 6 0 ____
textwrap 0.16.0 0 2234 73 0 ____
mc-sgx-core-types 0.7.0 0 2255 1 0 ____
p256 0.13.2 0 2545 25 0 ____
mc-sgx-dcap-types 0.7.0 0 2587 0 12 ____
rand 0.8.5 0 5273 64 32 ____
der 0.7.6 0 7820 24 0 ____
serde 1.0.164 0 10262 49 5 CB__
serde_json 1.0.96 0 12157 41 7 CB__

@codecov
Copy link

codecov bot commented Jun 30, 2023

Codecov Report

Merging #124 (66350fa) into main (bcc70c6) will increase coverage by 0.24%.
The diff coverage is 97.81%.

@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
+ Coverage   98.11%   98.35%   +0.24%     
==========================================
  Files          12       12              
  Lines        3450     3657     +207     
==========================================
+ Hits         3385     3597     +212     
+ Misses         65       60       -5     
Impacted Files Coverage Δ
verifier/src/lib.rs 96.11% <0.00%> (ø)
verifier/src/identity.rs 98.94% <98.24%> (+4.20%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nick-mobilecoin nick-mobilecoin requested review from eranrund and dolanbernard and removed request for a team June 30, 2023 15:34
@nick-mobilecoin nick-mobilecoin merged commit dbaf9af into main Jun 30, 2023
20 of 21 checks passed
@nick-mobilecoin nick-mobilecoin deleted the nick/identity-verifier branch June 30, 2023 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Large PRs
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants