http_client: Fixing bug in retry after a close call
After a close call httpclient is not starting with fresh socket to
restart the connection. Send failure is observed in cases where new
connection is started from doResolve.Calling restartConnection instead
of doResolve did fix the issue.
Tested By:
Running developer test on use cases such as redfish aggregation where
number of retries are smaller.
Change-Id: I12f6a73fbafd14f482807f34ffa1e02fad944fc1
Signed-off-by: Abhilash Raju <abhilash.kollam@gmail.com>
diff --git a/http/http_client.hpp b/http/http_client.hpp
index 0bfb381..107f943 100644
--- a/http/http_client.hpp
+++ b/http/http_client.hpp
@@ -706,7 +706,7 @@
// Server is not keep-alive enabled so we need to close the
// connection and then start over from resolve
conn->doClose();
- conn->doResolve();
+ conn->restartConnection();
}
return;
}
@@ -760,7 +760,7 @@
{
BMCWEB_LOG_DEBUG("Reusing existing connection {}",
commonMsg);
- conn->doResolve();
+ conn->restartConnection();
}
return;
}