sws: allow connection-monitor to log all disconnects

Remove the suppression of connmon after the first disconnect event. The
connmon flag is set per-test via the 'connection-monitor' server command
and should remain active for the lifetime of that test's connections to
properly detect connection reuse failures.

The suppression was introduced in 510fdad to work around a connection
reuse regression that has since been resolved by the credentials
refactoring in 8f71d0f. With the underlying issue fixed, restoring full
disconnect logging strengthens tests like 338 to catch future
regressions in connection reuse logic.

Fixes #22158
Closes #22159
This commit is contained in:
Samuel Dainard 2026-06-24 19:53:52 +00:00 committed by Daniel Stenberg
parent 4ce309d968
commit 5232c16208
No known key found for this signature in database
GPG key ID: 5CC908FDB71E12C2

View file

@ -2327,7 +2327,6 @@ static int test_sws(int argc, const char *argv[])
if(req->connmon) {
const char *keepopen = "[DISCONNECT]\n";
storerequest(keepopen, strlen(keepopen), REQUEST_DUMP_FILENAME);
req->connmon = FALSE;
}
if(!req->open)