Skip to content

chore: move attestation-provider-server crate from proxy repo#17

Open
steph-rs wants to merge 2 commits into
flashbots:mainfrom
steph-rs:steph-rs/chore/move-attestation-provider-server
Open

chore: move attestation-provider-server crate from proxy repo#17
steph-rs wants to merge 2 commits into
flashbots:mainfrom
steph-rs:steph-rs/chore/move-attestation-provider-server

Conversation

@steph-rs

@steph-rs steph-rs commented Apr 2, 2026

Copy link
Copy Markdown

This PR moves attestation-provider-server crate from proxy repo.
As discussed in flashbots/attested-tls-proxy#156.

Changes:

  • replaced attested-tls-proxy dependency with direct attestation path dependency
  • fmt
  • used tokio from workspace dependencies

@ameba23 ameba23 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great! Thank you.

Will also get @0x416e746f6e to take a look.

For context, this moves the crate as-is over from the attested-tls-proxy repo, as it is more related to the contents of this repo than to the proxy.

@ameba23 ameba23 requested a review from 0x416e746f6e April 3, 2026 09:12

@0x416e746f6e 0x416e746f6e left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks for this contribution.

small nit: can the provider server be made to fail with error if attestation type detection fails?

Comment on lines +14 to +15
attestation = { path = "../attestation" }
tokio.workspace = true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

suggestion:

Suggested change
attestation = { path = "../attestation" }
tokio.workspace = true
attestation = { workspace = true }
tokio = { workspace = true }

@0x416e746f6e

Copy link
Copy Markdown
Member

p.s. also, could you please rebase this PR @steph-rs?

@steph-rs

steph-rs commented May 7, 2026

Copy link
Copy Markdown
Author

yeah, I'll do fixes and rebase in a few days. happy to help

@ameba23

ameba23 commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

yeah, I'll do fixes and rebase in a few days. happy to help

@steph-rs gentle reminder about this. i can take it on if you are busy with other stuff these days.

@steph-rs

steph-rs commented Jun 9, 2026

Copy link
Copy Markdown
Author

I apologize for the delay. I've been busy with a lot of things. I remember this PR, and I'm going to do it today, I promise.

@ameba23

ameba23 commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

I apologize for the delay. I've been busy with a lot of things.

No worries, we know how it is 😉

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.

3 participants