-
Notifications
You must be signed in to change notification settings - Fork 98
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 implementation of AESM client for SGX #222
Conversation
0eeeea2
to
cf9c8e5
Compare
aesm-client/src/lib.rs
Outdated
#[derive(Default, Debug, Clone)] | ||
pub struct AesmClient { | ||
inner: imp::AesmClient | ||
} | ||
|
||
// TODO: Find a better alternative to removing Clone and Default trait |
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.
use cfg_attr
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.
Delete comment
465f139
to
3a0c541
Compare
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.
Ok this looks better, some small cleanup needed.
.travis.yml
Outdated
@@ -55,4 +56,4 @@ matrix: | |||
- cargo test --verbose -p sgxs | |||
- cargo test --verbose -p sgxs-loaders | |||
- cargo test --verbose -p sgxs-tools --features pe2sgxs --bin isgx-pe2sgx -Z package-features | |||
|
|||
- cargo build --verbose -p aesm-client |
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.
Duplicate of line 49?
aesm-client/src/lib.rs
Outdated
#[derive(Default, Debug, Clone)] | ||
pub struct AesmClient { | ||
inner: imp::AesmClient | ||
} | ||
|
||
// TODO: Find a better alternative to removing Clone and Default trait |
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.
Delete comment
} | ||
|
||
// FIXME: remove conditional compilation after resolving https://github.com/fortanix/rust-sgx/issues/31 | ||
#[cfg(not(target_env = "sgx"))] |
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.
Just do this conditional inline (as in get_quote
). You can apply #[allow(unused_mut)]
on the binding.
This commit allows for using the AESM client from inside the enclave Add building of aesm for SGX target on linux in CI Add tests to build aesm-client during CI for all platforms Added a common aesm_proto mod for sgx and unix
bors r+ |
Build succeeded
|
The implementation of AESM for SGX takes in a TcpStream for its constructor.
It omits the get_launch_token functionality