From f44f56ea2cd7fa8148fd84dd9cfa77744a250c11 Mon Sep 17 00:00:00 2001 From: "Namhyeon, Go" Date: Sun, 17 Aug 2025 20:43:33 +0900 Subject: [PATCH] Refactor HTTP client usage in ResourceServer Replaced local HttpClient instantiation with the class-level _httpClient for sending requests. This improves resource management and consistency in HTTP operations. --- .../WelsonJS.Launcher/ResourceServer.cs | 41 +++++++++---------- 1 file changed, 19 insertions(+), 22 deletions(-) diff --git a/WelsonJS.Toolkit/WelsonJS.Launcher/ResourceServer.cs b/WelsonJS.Toolkit/WelsonJS.Launcher/ResourceServer.cs index 8c55717..f77a842 100644 --- a/WelsonJS.Toolkit/WelsonJS.Launcher/ResourceServer.cs +++ b/WelsonJS.Toolkit/WelsonJS.Launcher/ResourceServer.cs @@ -164,30 +164,27 @@ namespace WelsonJS.Launcher try { - using (var client = new HttpClient()) + HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, url); + var ua = context?.Request?.UserAgent; + if (!string.IsNullOrEmpty(ua)) { - HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, url); - var ua = context?.Request?.UserAgent; - if (!string.IsNullOrEmpty(ua)) - { - request.Headers.UserAgent.ParseAdd(ua); - } - HttpResponseMessage response = await client.SendAsync(request); - - if (!response.IsSuccessStatusCode) - { - _logger?.Error($"Failed to serve blob. URL: {url}, Status: {response.StatusCode}"); - return false; - } - - data = await response.Content.ReadAsByteArrayAsync(); - mimeType = response.Content.Headers.ContentType?.MediaType ?? _defaultMimeType; - - ServeResource(context, data, mimeType); - _ = TrySaveCachedBlob(path, data, mimeType); - - return true; + request.Headers.UserAgent.ParseAdd(ua); } + HttpResponseMessage response = await _httpClient.SendAsync(request); + + if (!response.IsSuccessStatusCode) + { + _logger?.Error($"Failed to serve blob. URL: {url}, Status: {response.StatusCode}"); + return false; + } + + data = await response.Content.ReadAsByteArrayAsync(); + mimeType = response.Content.Headers.ContentType?.MediaType ?? _defaultMimeType; + + ServeResource(context, data, mimeType); + _ = TrySaveCachedBlob(path, data, mimeType); + + return true; } catch (Exception ex) {