Opened 21 months ago

Last modified 2 weeks ago

#6371 enhancement new

Support native Windows trusted CA database for SSL certificate validation

Reported by: itamar Owned by:
Priority: normal Milestone:
Component: core Keywords: review ssl
Cc: Branch:
Author: Launchpad Bug:

Description

This was originally part of #5446, where Glyph wrote:

On Windows - and this is purely from a quick glance at the reference documentation, so take it with a grain of salt - I believe the right way to do this is to use CertOpenSystemStore with the string "CA", or possibly "ROOT", or maybe both, and then do CertEnumCertificatesInStore or maybe just PFXExportCertStoreEx to dump the certs into a format we can import into OpenSSL.

Attachments (3)

windows_trust_root.patch (4.4 KB) - added by aronbierbaum 2 weeks ago.
windows_trust_root2.patch (4.4 KB) - added by aronbierbaum 2 weeks ago.
windows-trust-root-6371-2.patch (4.4 KB) - added by aronbierbaum 2 weeks ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 21 months ago by exarkun

There is a Python library to help out with this too:

https://bitbucket.org/tiran/wincertstore

This may be easier than writing C, or Cython, or using ctypes or cffi. Or maybe not, I haven't investigated much. But it's something to look at I guess.

comment:2 Changed 9 months ago by itamar

wincertstore is written with ctypes.

comment:3 Changed 9 months ago by aronbierbaum

I have been working on an implementation that uses for ctypes also. In the spirit of not duplicating effort, is anyone else also working on a solution?

comment:4 Changed 9 months ago by itamar

Assuming wincertstore does what it says it does, presumably that's who you should talk to. Unless you want to add something to Twisted, in which case using wincertstore seems like a good start.

Changed 2 weeks ago by aronbierbaum

comment:5 Changed 2 weeks ago by aronbierbaum

  • Keywords review ssl added

I have added an initial pass at adding support for certificate verification on Windows using the wincertstore package. Please provide feedback and required changes.

Changed 2 weeks ago by aronbierbaum

Changed 2 weeks ago by aronbierbaum

Note: See TracTickets for help on using tickets.