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

Only SSL checks for *.sha2017.org #97

Open
Roosted7 opened this issue Jul 22, 2017 · 7 comments
Open

Only SSL checks for *.sha2017.org #97

Roosted7 opened this issue Jul 22, 2017 · 7 comments
Labels

Comments

@Roosted7
Copy link
Member

Can we include Let's encrypt?

@annejan
Copy link
Member

annejan commented Jul 22, 2017

Yes, but how do we 'know' to use that (root) certificate?

The whole 'just *.sha2017.org' bit of code enforces certificate validation only for the domain since we know what root CA it uses . .

@Roosted7
Copy link
Member Author

Roosted7 commented Jul 22, 2017

Is it not possible to do the same for Let's encrypt? Root certs from LE

This kinda crypto is not my area of expertise, I'm justing doodling out what would be nice to see

@basvs
Copy link

basvs commented Jul 24, 2017

You cannot add only part of the root CAs, because you don't know which domain uses which root CA.

@basvs
Copy link

basvs commented Jul 24, 2017

I am thinking about a 'ssl-certs' egg/addon, which manages a path with tls certificates (/etc/tls/ ?)

And then could fix the tls library to automatically load root certificates from that path.

@Roosted7
Copy link
Member Author

Is this still a wontfix with @basvs comments?

@annejan
Copy link
Member

annejan commented Jul 30, 2017

Well . . if there is a clear path on how to do this.
But I don't really have a nice clean idea on how to set this up (calls wise) . .

Code wise it should not be very hard, but I don't really see any way to do this and maintain compatibility with other things.

@basvs
Copy link

basvs commented Jul 30, 2017

With mbedtls_ssl_conf_verify() you can add your own callback to verify certificates.

If the library then cannot verify a certificate, this method is called. We can lookup the root certificate, then search on the filesystem for a file "/etc/certificates/".md5(cert.subject) and call the mbedtls_x509_crt_verify() method again, but now with the extra root-certificate and without the callback-handler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants