[SERVER-56977] Cleanup: tighter const safety declarations in transport layer and SSL manager Created: 14/May/21 Updated: 24/May/21 Resolved: 24/May/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Andrew Shuvalov (Inactive) | Assignee: | Andrew Shuvalov (Inactive) |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Participants: | |||||
| Linked BF Score: | 131 | ||||
| Description |
|
While working on BF ticket I found the code will be a lot more easier to reason about if the methods and fields in TransportLayer, SSLManagerInterface and ASIOSession are changed to const. The main idea of this change is to prove that creating new connections on transport is not mutating the transport itself. I will not enumerate details because the change is already made and I will post the CR shortly. |