refactor: replaced bytes.NewBuffer with bytes.NewBufferString (#473)

This PR change one function of bytes package with a similar one, as result code produce less allocations on heap.
Clivern · Oct 20, 2021 · 00f7ce3 · 00f7ce3
1 parent 422e2bb
commit 00f7ce3
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/cmd/agent.go b/cmd/agent.go
@@ -53,7 +53,7 @@ var agentCmd = &cobra.Command{
 		}
 
 		viper.SetConfigType("yaml")
-		err = viper.ReadConfig(bytes.NewBuffer([]byte(configParsed)))
+		err = viper.ReadConfig(bytes.NewBufferString(configParsed))
 
 		if err != nil {
 			panic(fmt.Sprintf(

diff --git a/core/service/http.go b/core/service/http.go
@@ -66,7 +66,7 @@ func (h *HTTPClient) Post(ctx context.Context, endpoint string, data string, par
 		return nil, err
 	}
 
-	req, _ := http.NewRequest("POST", endpoint, bytes.NewBuffer([]byte(data)))
+	req, _ := http.NewRequest("POST", endpoint, bytes.NewBufferString(data))
 
 	req = req.WithContext(ctx)
 
@@ -96,7 +96,7 @@ func (h *HTTPClient) Put(ctx context.Context, endpoint string, data string, para
 		return nil, err
 	}
 
-	req, _ := http.NewRequest("PUT", endpoint, bytes.NewBuffer([]byte(data)))
+	req, _ := http.NewRequest("PUT", endpoint, bytes.NewBufferString(data))
 
 	req = req.WithContext(ctx)
 
@@ -126,7 +126,7 @@ func (h *HTTPClient) Patch(ctx context.Context, endpoint string, data string, pa
 		return nil, err
 	}
 
-	req, _ := http.NewRequest("PATCH", endpoint, bytes.NewBuffer([]byte(data)))
+	req, _ := http.NewRequest("PATCH", endpoint, bytes.NewBufferString(data))
 
 	req = req.WithContext(ctx)
 

diff --git a/pkg/loader.go b/pkg/loader.go
@@ -49,7 +49,7 @@ func LoadConfigs(path string) error {
 
 	viper.SetConfigType("yaml")
 
-	viper.ReadConfig(bytes.NewBuffer([]byte(data1)))
+	viper.ReadConfig(bytes.NewBufferString(data1))
 
 	viper.SetDefault("app.name", "x-x-x-x")
 	viper.SetDefault("role", "tower")