[SERVER-36866] Fix switch in Secure Transport recordTLSVersion() Created: 24/Aug/18 Updated: 23/Sep/18 Resolved: 27/Aug/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Sara Golemon | Assignee: | Mark Benvenuto |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Operating System: | ALL | ||||||||
| Participants: | |||||||||
| Description |
|
The current set of cases in SecureTransport's recordTLSVersion() explicitly leaves out kTLSProtocol13 because that enum value isn't defined in older OSX headers. Unfortunately, this makes compiling with newer OSX headers broken since it's an unhandled enum value. We should either use a `default:` case to ignore it (it's already being ignored anyway), or find the right #ifdef to determine if the enum is defined and conditionally capture it. |