-
Notifications
You must be signed in to change notification settings - Fork 0
Proof-of-concept for adding attested TLS server #20
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
base: main
Are you sure you want to change the base?
Conversation
| None | ||
| }; | ||
|
|
||
| let server = if let Some(attested_tls_server) = attested_tls_server { |
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.
question:
this doesn't seem to implement nested TLS (self-signed CA with attestation primitives stream inside public CA rooted TLS stream). instead it's either one or the other. am I correct?
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.
No, no. I didn't get that far. I first just want to establish that this works at all.
|
i can confirm i successfully rproxied a request with this. But one thing which bit me is that Next steps: Ideally i would like to write a unit test which demonstrates using attested-tls, To be able to iterate a bit faster and know it still works. |
This shows where how i would imagine to integrate attested-tls server into the rproxy http proxy server.
Its proof of concept in that there is no error handling, and currently does not take any attestation configuration - just demonstrates how the pieces could fit together.
Also note this does not [currently] add nested TLS - it simply uses the TLS key and certificate from the config for attested TLS.
This compiles but is not tested. @0x416e746f6e whats the best way to test this - do you have some kind of test setup for rproxy?
I did a
cargo updateto get around some dependency conflicts. If you prefer i only update the directly relevant crates let me know.Also worth noting - this introduces a dependency on openssl. Its not used by attested-tls directly but when retrieving collateral from PCCS during attestation verification via
reqwest.Still missing:
AttestationGeneratortdx-quote-providerattested-tls-proxy#100