Timeouts and HttpCompletionOption.ResponseHeadersRead updated

Author: mendozagit

Common/Http/SatService.cs

@@ -81,8 +81,11 @@ protected async Task<SatResponse> SendRequestAsync(string url, string action, st
         {
             LogRequest(request, url, action, payload);
 
+            // Use ResponseHeadersRead to complete as soon as headers are received, allowing early error detection
+            // This prevents waiting for the entire response body before detecting HTTP errors, which can help
+            // reduce timeout issues when the SAT server is slow to generate/transmit large ZIP packages
             using var response =
-                await _httpClient.SendAsync(request, HttpCompletionOption.ResponseContentRead, cancellationToken);
+                await _httpClient.SendAsync(request, HttpCompletionOption.ResponseHeadersRead, cancellationToken);
             var responseContent = await response.Content.ReadAsStringAsync(cancellationToken);
 
             LogResponse(response, responseContent, url, action);

DownloaderExtensions.cs

@@ -171,11 +171,26 @@ public static TEnum ToEnumElement<TEnum>(this string code) where TEnum : struct,
     /// <returns>The service collection for chaining</returns>
     public static IServiceCollection AddXmlDownloader(this IServiceCollection services)
     {
-        // Register HttpClient factory for SAT services
-        services.AddHttpClient<IAuthService, AuthService>();
-        services.AddHttpClient<IQueryService, QueryService>();
-        services.AddHttpClient<IVerifyService, VerifyService>();
-        services.AddHttpClient<IDownloadService, DownloadService>();
+        // Register HttpClient factory for SAT services with appropriate timeouts
+        services.AddHttpClient<IAuthService, AuthService>(client =>
+        {
+            client.Timeout = TimeSpan.FromMinutes(3);
+        });
+        
+        services.AddHttpClient<IQueryService, QueryService>(client =>
+        {
+            client.Timeout = TimeSpan.FromMinutes(3);
+        });
+        
+        services.AddHttpClient<IVerifyService, VerifyService>(client =>
+        {
+            client.Timeout = TimeSpan.FromMinutes(3);
+        });
+        
+        services.AddHttpClient<IDownloadService, DownloadService>(client =>
+        {
+            client.Timeout = TimeSpan.FromMinutes(7);
+        });
 
         // Register non-HTTP services as scoped
         services.AddScoped<IFileStorageService, FileStorageService>();