Skip to content
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

visibility of OwnedTrustAnchor #377

Closed
dingelish opened this issue Jun 15, 2020 · 5 comments
Closed

visibility of OwnedTrustAnchor #377

dingelish opened this issue Jun 15, 2020 · 5 comments

Comments

@dingelish
Copy link

Hi ctz,

Our team is using rustls to terminate TLS connections under the spiffe framework. rustls works pretty well and we all like it! The only problem in our implementation is that we have to use OwnedTrustAnchor::to_trust_anchor to verify certificates, while OwnedTrustAnchor is not a pubed structure. We're using an internal fork of rustls and wonder if you can make it public. It'll be the best if we can use it directly from crates.io and we don't mind feature gating. Really appreciate your help. Thanks!

Best,
Yu

@ctz
Copy link
Member

ctz commented Jun 20, 2020

I've added a test in 1b99071 that checks that this is usable outside the crate already -- could you look at the test and tell me if I've misunderstood what you're asking for?

@ctz ctz closed this as completed Jul 18, 2020
@nightkr
Copy link

nightkr commented Sep 18, 2020

@ctz Both the struct and method are pub, but the struct is unreachable (since the anchors module isn't pub, and OwnedTrustAnchor isn't covered by the same pub use as RootCertStore.

@blaggacao
Copy link

@ctz @dingelish Would it be conceivable to add yours as a sort of contrib implementation of the SPIFFE protocol to rustls?

@dingelish
Copy link
Author

@ctz @dingelish Would it be conceivable to add yours as a sort of contrib implementation of the SPIFFE protocol to rustls?

@blaggacao sgtm. thanks!

@blaggacao
Copy link

blaggacao commented Oct 13, 2020

Breadcrumb trail — See also discussion at: https://www.mail-archive.com/openssl-users@openssl.org/msg88596.html

And probably: rwf2/Rocket#1448

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

No branches or pull requests

4 participants