-
Notifications
You must be signed in to change notification settings - Fork 69
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
Is Wreq suitable for HTTPS applications? #82
Comments
It seems that Wreq (correctly) rejects self-signed certificates. I ran this example code
replacing
I think that the TODO.md simply needs to be updated. |
TryTLS test tool (https://github.com/ouspg/trytls) has some tests for HTTPS (mostly certificate handling). I have contributed Wreq stub: (https://github.com/ouspg/trytls/tree/master/stubs/haskell-wreq) Currently running TryTLS tests for Wreq looks like: $ trytls -t .https.all_tests -- docker run --rm test-wreq
PASS badssl(False, 'expired')
FAIL badssl(False, 'wrong.host')
PASS badssl(False, 'self-signed')
PASS badssl(True, 'sha256')
PASS badssl(True, '1000-sans')
PASS badssl(True, '10000-sans')
PASS badssl(False, 'incomplete-chain')
FAIL badssl(False, 'pinning-test')
PASS badssl(False, 'superfish')
PASS badssl(False, 'edellroot')
PASS badssl(False, 'dsdtestprovider')
SKIP local(True, 'localhost', callback=<function https_callback at 0x10f069230>)
SKIP local(False, 'nothing', callback=<function https_callback at 0x10f069230>) That one fail with "wrong.host" is reported in #84. With this set of tests, Wreq looks good for HTTPS connections. |
Wow, this is great work! Thanks for doing this! |
One of the items in the TODO is
Does this mean that anyone can create (self-sign) a certificate for e.g. www.google.com and Wreq will accept this certificate?
The text was updated successfully, but these errors were encountered: