Refactor - simplify code

api/rest/http.go

@@ -19,37 +19,31 @@ func NewHttpClient(baseURL string) *HttpClient {
 
 func (c *HttpClient) SendGet(url string, data, out any) error {
 	fmt.Printf("Sending req to: %s%s\n", c.baseURL, url)
-	res, err := c.sendRequest(c.baseURL, url, http.MethodGet, data)
+	res, err := c.sendRequest(url, http.MethodGet, data)
 	if err != nil {
 		return err
 	}
 
 	decoder := json.NewDecoder(res.Body)
-	err = decoder.Decode(&out)
+	defer res.Body.Close()
+
+	return decoder.Decode(&out)
+}
+
+func (c *HttpClient) SendPost(url string, data, out any) error {
+	res, err := c.sendRequest(url, http.MethodPost, data)
 	if err != nil {
 		return err
 	}
 
-	return nil
-}
-
-func (c *HttpClient) SendPost(url string, data, out any) (any, error) {
-	res, err := c.sendRequest(c.baseURL, url, http.MethodPost, data)
-	if err != nil {
-		return nil, err
-	}
-
 	decoder := json.NewDecoder(res.Body)
-	err = decoder.Decode(out)
+	defer res.Body.Close()
-	if err != nil {
-		return nil, err
-	}
 
-	return out, nil
+	return decoder.Decode(&out)
 }
 
-func (c *HttpClient) sendRequest(baseURL, actionURL, method string, data any) (*http.Response, error) {
+func (c *HttpClient) sendRequest(url, method string, data any) (*http.Response, error) {
-	apiUrl := baseURL + actionURL
+	apiUrl := c.baseURL + url
 
 	tr := &http.Transport{
 		TLSClientConfig: &tls.Config{InsecureSkipVerify: true}, // FIXME dev mode
@@ -74,7 +68,6 @@ func (c *HttpClient) sendRequest(baseURL, actionURL, method string, data any) (*
 	if err != nil {
 		return nil, err
 	}
-	defer res.Body.Close()
 
 	return res, nil
 }

api/rest/identity.go

@@ -17,7 +17,6 @@ func (a *IdentityAPI) CheckAccess(url, srv string) (*identity.CheckAccessRespons
 	req := &identity.CheckAccessRequestDTO{URL: url, Service: srv}
 	res := new(identity.CheckAccessResponseDTO)
 
-	// targetURL := fmt.Sprintf("/v1/access?url=%s&srv=%s", url, srv)
 	if err := a.httpClient.SendGet("/v1/access", req, res); err != nil {
 		return nil, err
 	}