Skip to content

Bump attested-tls repo depdencies and pin them - #193

Merged
ameba23 merged 1 commit into
mainfrom
peg/bump-attested-tls-crates-84
Oct 30, 2026
Merged

ameba23 merged 1 commit into
mainfrom
peg/bump-attested-tls-crates-84

Conversation

@ameba23

@ameba23 ameba23 commented Oct 30, 2026

Copy link
Copy Markdown
Collaborator

This bumps dependencies from the flashbots/attested-tls repo and updates for very minor api change. It also pins those dependencies to a specific commit (current head of main).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The DCAP and cryptographic dependency upgrades require human validation because upstream code is unavailable and mocked tests do not establish real TDX/Azure attestation compatibility.

No open issues 🎉

What changed in this PR

Updates the attested TLS dependencies and pins them to commit ca2a75d3de428543947cf1272b425c5ca2c534d3, adapting local error handling to the upstream API change.

Changes:

  • Pins attestation and pccs to the same upstream revision.
  • Uses AttestedTlsError for report-input errors and adds tests for empty and malformed certificate chains.
  • Refreshes the lockfile with updated transitive dependencies.
File Description
crates/​attested-tls/​src/​lib.rs Updates error types, adds X.509 parse-error conversion, and tests invalid certificate chains.
crates/​attested-tls/​Cargo.toml Pins production and test attestation dependencies.
crates/​attested-tls-proxy/​Cargo.toml Pins the pccs dependency.
Cargo.lock Records pinned Git revisions and updated dependency versions.

Review effort: Balanced

@ameba23
ameba23 merged commit 722bdcf into main Oct 30, 2026
3 checks passed
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