Info
In this line it is assumed that the response can be parsed as a StatusWith. However in case of errors it will be a simple Status.
This will trigger an unhandled std::exception crashing the server. A solution would be to adopt the approach currently followed to handle exceptions when trying to parse a StatusWith.
Top User Comments
pierlauro.sciarelli commented on Mon, 3 Apr 2023 17:08:45 +0000:
Closing the ticket because it turns out the issue was triggered by this test that was explicitly constructing a Status rather than a StatusWith. This was possible because the BalancerStreamActionResponse is a variant hence it is totally possible to compile/execute with a wrongly typed response.
SERVER-75520 corrected the code that was handling the test-induced issue.
