fix: httpClient being created for every request

[petruki]

This pull request introduces improvements to the HTTP client management and request handling logic in the client package. The main changes add thread-safe, lazy initialization and caching for the HTTP client, and ensure safer handling of nil entries in request payloads.

HTTP client management improvements:

• Added a mutex (httpClientMu) and a cached HTTP client field (httpClient_) to the Client struct to support thread-safe, lazy initialization and reuse of the http.Client instance. (client.go client.goR21-R23)
• Modified the httpClient() method to use the mutex and cache, ensuring only one HTTP client is created and reused across requests. (remote.go [1] [2]
• Imported the net/http package to support the new HTTP client logic. (client.go client.goR4)

Request payload safety:

• Updated the checkCriteria method to ensure that the entries field in the request payload is always a valid (non-nil) slice, preventing potential issues when the field is nil. (remote.go remote.goR97-R106)

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud