Refactor - simplify code

api/rest/http.go

@@ -25,27 +25,21 @@ func (c *HttpClient) SendGet(url string, data, out any) error {
 	}
 
 	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(url, http.MethodPost, data)
-	if err != nil {
-		return nil, err
-	}
-
 	decoder := json.NewDecoder(res.Body)
-	err = decoder.Decode(out)
-	if err != nil {
-		return nil, err
-	}
+	defer res.Body.Close()
 
-	return out, nil
+	return decoder.Decode(&out)
 }
 
 func (c *HttpClient) sendRequest(url, method string, data any) (*http.Response, error) {
@@ -74,7 +68,6 @@ func (c *HttpClient) sendRequest(url, method string, data any) (*http.Response,
 	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
 	}