Skip to content

Conversation

@kvinwang
Copy link
Collaborator

No description provided.

Evrard-Nil and others added 24 commits January 14, 2026 16:11
…igurable-

Make DNS TXT record TTL configurable.
This ensures the logo renders properly in dark mode environments
like GitHub's dark theme.

Closes #439

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
fix: add white background to banner image for dark mode
Refactor attestation for multi-provider support
Previously the code was reading the entire sys-config.json file as the
config, but it should only read the vm_config field within it.

Extracted read_vm_config() function that properly parses SysConfig and
returns the vm_config field.
Fix vm_config loading from sys-config.json
The TDX driver does not support concurrent access. Add global mutex
locks at both tdx-attest and dstack-attest layers to prevent race
conditions:

- tdx-attest: Lock in get_quote() to protect low-level TDX driver calls
- dstack-attest: Lock in quote_with_app_id() for future TEE environments
Remove the pccs_url parameter from the VerificationRequest struct to
prevent potential SSRF attacks where a malicious client could specify
an arbitrary URL. The PCCS URL is now only configurable via the server
configuration file.

Changes:
- Remove pccs_url field from VerificationRequest in types.rs
- Add pccs_url to CvmVerifier struct instead of passing per-request
- Update all CvmVerifier::new() calls to pass pccs_url from config
- Update README to reflect the configuration-only approach
Strip boot-time event log payloads (RTMR 0-2) from GetQuote, Attest, and
TdxQuote responses to reduce response size. Only digests are retained for
verification purposes; runtime events (RTMR3) keep full payloads.

Add documentation explaining event_log_verified semantics: RTMR3 events
have both digest and payload verified, while RTMR 0-2 events only have
digests verified through replay comparison.
@kvinwang kvinwang merged commit 590b8f3 into master Jan 21, 2026
1 of 4 checks passed
@kvinwang kvinwang deleted the pha-merge-upstream branch January 21, 2026 07:46
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.

4 participants