fix: Cancel the previous request before scheduling a new one in HTTP …

…resumer

The `http::Client()` class is designed to always have only one
HTTP request in progress. Thus, before scheduling a new request
using the same `http::Client` instance, cancel the previous
request to make sure everything is properly reset for the new
one.

Ticket: MEN-7810
Changelog: Fix download resuming to reset the HTTP state and
avoid repeatedly hitting the same error in case of a bad state

Signed-off-by: Vratislav Podzimek <[email protected]>

src/common/http/http_resumer.cpp

@@ -423,6 +423,9 @@ http::OutgoingRequestPtr DownloadResumerClient::RemainingRangeRequest() const {
 };
 
 error::Error DownloadResumerClient::ScheduleNextResumeRequest() {
+	// In any case, make sure the previous HTTP request is cancelled.
+	client_.Cancel();
+
 	auto exp_interval = retry_.backoff.NextInterval();
 	if (!exp_interval) {
 		return http::MakeError(