Refactor - simplify code

api/rest/basket.go

@@ -18,7 +18,7 @@ type BasketAPI struct {
 func (a *BasketAPI) GetBasket(basketID string) (*basket.GetBasketResponseDTO, error) {
 	req := &basket.GetBasketRequestDTO{BasketID: basketID}
 	res := new(basket.GetBasketResponseDTO)
-	if err := a.httpClient.SendGet("basket-svc", "/api/v1/basket", req, res); err != nil {
+	if err := a.httpClient.SendGet("/api/v1/basket", req, res); err != nil {
 		return nil, err
 	}
 
@@ -30,7 +30,7 @@ func (a *BasketAPI) GetBasketItems(basketID string) ([]*basket.GetBasketItemsRes
 	req := basket.GetBasketItemsRequestDTO{BasketID: basketID}
 	var res []*basket.GetBasketItemsResponseDTO
 
-	if err := a.httpClient.SendGet("basket-svc", url, req, &res); err != nil {
+	if err := a.httpClient.SendGet(url, req, &res); err != nil {
 		return nil, err
 	}
 

api/rest/http.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"crypto/tls"
 	"encoding/json"
+	"fmt"
 	"net/http"
 )
 
@@ -12,49 +13,44 @@ type HttpClient struct {
 	userAgent string
 }
 
-func NewHttpClient(apiURL string) *HttpClient {
-	return &HttpClient{baseURL: apiURL, userAgent: "internal-http-client"}
+func NewHttpClient(baseURL string) *HttpClient {
+	return &HttpClient{baseURL: baseURL, userAgent: "internal-http-client"}
 }
 
-func (c *HttpClient) SendGet(api, url string, data, out any) error {
-	res, err := c.sendRequest(api, url, http.MethodGet, data)
+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(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(api, url string, data, out any) (any, error) {
-	res, err := c.sendRequest(api, 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(api, url, method string, data any) (*http.Response, error) {
-	apiUrl := api + url // FIXME
+func (c *HttpClient) sendRequest(url, method string, data any) (*http.Response, error) {
+	apiUrl := c.baseURL + url
 
 	tr := &http.Transport{
-		TLSClientConfig: &tls.Config{InsecureSkipVerify: true}, // FIXME
+		TLSClientConfig: &tls.Config{InsecureSkipVerify: true}, // FIXME dev mode
 	}
 
 	// Create an HTTP client with the custom transport
-	client := &http.Client{Transport: tr} // FIXME
+	client := &http.Client{Transport: tr} // FIXME dev mode
 
 	json, err := json.Marshal(&data)
 	if err != nil {
@@ -72,7 +68,6 @@ func (c *HttpClient) sendRequest(api, url, method string, data any) (*http.Respo
 	if err != nil {
 		return nil, err
 	}
-	defer res.Body.Close()
 
 	return res, nil
 }

api/rest/identity.go

@@ -17,8 +17,7 @@ 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("identity-svc", "/v1/access", req, res); err != nil {
+	if err := a.httpClient.SendGet("/v1/access", req, res); err != nil {
 		return nil, err
 	}
 

api/rest/pricing.go

@@ -18,7 +18,7 @@ type PricingAPI struct {
 func (a *PricingAPI) GetProductPrice(productID int) (*pricing.ProductPriceResponseDTO, error) {
 	url := fmt.Sprintf("/api/v1/product/%d", productID)
 	res := new(pricing.ProductPriceResponseDTO)
-	if err := a.httpClient.SendGet("pricing-svc", url, nil, res); err != nil {
+	if err := a.httpClient.SendGet(url, nil, res); err != nil {
 		return nil, err
 	}