[CDRIVER-1409] Test that topology scanner is still async with TLS Created: 22/Jul/16 Updated: 25/Jan/18 Resolved: 25/Jan/18 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | tests |
| Affects Version/s: | 1.4.0 |
| Fix Version/s: | 1.10.0 |
| Type: | New Feature | Priority: | Minor - P4 |
| Reporter: | A. Jesse Jiryu Davis | Assignee: | A. Jesse Jiryu Davis |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Epic Link: | Native TLS and SCRAM-SHA-1 | ||||||||||||||||||||||||
| Description |
|
Use the mock_server_t for end-to-end testing of the driver's topology scanner, especially that it is still async – ensuring we avoided introducing blocking code into the scanner was a focus of the native TLS code reviews ( Now that we use SSL_shutdown on OpenSSL connections ( |
| Comments |
| Comment by A. Jesse Jiryu Davis [ 25/Jan/18 ] |
|
The mock server can accept SSL connections with OpenSSL, so we test that the scanner is async with OpenSSL builds. I've manually verified that the scanner is async with Windows SChannel, but I won't implement the server side of SSL with SChannel in the mock server. Apple Secure Transport is still blocking; that's ok so long as programmers only use Mac OS X for development, not deployment. |
| Comment by Githook User [ 06/Jan/18 ] |
|
Author: {'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis', 'email': 'jesse@mongodb.com'}Message: |
| Comment by Githook User [ 05/Jan/18 ] |
|
Author: {'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis', 'email': 'jesse@mongodb.com'}Message: |
| Comment by A. Jesse Jiryu Davis [ 09/Aug/16 ] |
|
I have a test in my local checkout that proves what we want to prove. It works with OpenSSL, and works with Secure Transport (Apple) for up to 8 mock servers, but hangs with 9. Haven't tried Secure Channel (Windows) at all. |
| Comment by Githook User [ 09/Aug/16 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Revert " This reverts commit 8642ac1a6dd26cc265fbd25c35d7134a21deb0ae. |
| Comment by Githook User [ 31/Jul/16 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: BIO_get_shutdown doesn't do what I thought: it doesn't check whether the It was only this test that failed because the driver only calls |
| Comment by Githook User [ 27/Jul/16 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: |
| Comment by Githook User [ 22/Jul/16 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Also detect remote TLS shutdown: Only matters for the server sockets we |