Skip to content

Add TDX quote provider service #200

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

Merged
merged 3 commits into from
Jul 16, 2025
Merged

Add TDX quote provider service #200

merged 3 commits into from
Jul 16, 2025

Conversation

avalonche
Copy link
Contributor

@avalonche avalonche commented Jul 12, 2025

πŸ“ Summary

Add http server that provides TDX quotes. Removes TDX dependency and flashtestations feature in op-rbuilder


βœ… I have completed the following steps:

  • Run make lint
  • Run make test
  • Added tests (if applicable)

@avalonche avalonche force-pushed the tdx-provider branch 2 times, most recently from c36e818 to 5640f38 Compare July 14, 2025 16:43
Copy link

@Ruteri Ruteri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

generating attestations lgtm

))
}
// TODO: replace with real attestation provider
Box::new(DebugAttestationProvider::new(
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably should be called "remote" provider or something

#[derive(Debug, Clone)]
pub struct ServerConfig {
/// Address to listen on
pub listen_addr: SocketAddr,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess ideally we'd be listening on either a socket or http laddr

@avalonche avalonche merged commit f0636ba into main Jul 16, 2025
4 checks passed
@avalonche avalonche deleted the tdx-provider branch July 16, 2025 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants