Closed d-w-moore closed 2 months ago
This is ready for review.
(Tests should include evaluating whether the new code works for SSL sockets.)
The SSL test for this should come as part of a different PR. Reason being : it depends on some test infrastructure that is not merged for this release and may indeed have to wait for a subsequent one.
Are the open conversations handled? It looked like they were last time I checked, but they have not been resolved, so I wanted to make sure.
Are the open conversations handled? It looked like they were last time I checked, but they have not been resolved, so I wanted to make sure.
Yes indeed-y.
Needs a test or two.
(Tests should include evaluating whether the new code works for SSL sockets.)