code improvements

client.go

@@ -594,7 +594,6 @@ func (c *Client) execute(req *Request) (*Response, error) {
 		}
 	}
 
-	// Do improvement here
 	c.mutex.Lock()
 
 	if req.proxyURL != nil {
@@ -609,6 +608,8 @@ func (c *Client) execute(req *Request) (*Response, error) {
 	c.httpClient.Transport = c.transport
 	resp, err := c.httpClient.Do(req.RawRequest)
 
+	c.mutex.Unlock()
+
 	response := &Response{
 		Request:     req,
 		RawResponse: resp,
@@ -629,8 +630,6 @@ func (c *Client) execute(req *Request) (*Response, error) {
 		response.size = int64(len(response.body))
 	}
 
-	c.mutex.Unlock()
-
 	// Apply Response middleware
 	for _, f := range c.afterResponse {
 		err = f(c, response)

resty_test.go

@@ -1033,9 +1033,9 @@ func TestDetectContentTypeForPointerWithSlice(t *testing.T) {
 	defer ts.Close()
 
 	users := &[]ExampleUser{
-		ExampleUser{FirstName: "firstname1", LastName: "lastname1", ZipCode: "10001"},
-		ExampleUser{FirstName: "firstname2", LastName: "lastname3", ZipCode: "10002"},
-		ExampleUser{FirstName: "firstname3", LastName: "lastname3", ZipCode: "10003"},
+		{FirstName: "firstname1", LastName: "lastname1", ZipCode: "10001"},
+		{FirstName: "firstname2", LastName: "lastname3", ZipCode: "10002"},
+		{FirstName: "firstname3", LastName: "lastname3", ZipCode: "10003"},
 	}
 
 	resp, err := dclr().
@@ -1080,9 +1080,9 @@ func TestDetectContentTypeForSlice(t *testing.T) {
 	defer ts.Close()
 
 	users := []ExampleUser{
-		ExampleUser{FirstName: "firstname1", LastName: "lastname1", ZipCode: "10001"},
-		ExampleUser{FirstName: "firstname2", LastName: "lastname3", ZipCode: "10002"},
-		ExampleUser{FirstName: "firstname3", LastName: "lastname3", ZipCode: "10003"},
+		{FirstName: "firstname1", LastName: "lastname1", ZipCode: "10001"},
+		{FirstName: "firstname2", LastName: "lastname3", ZipCode: "10002"},
+		{FirstName: "firstname3", LastName: "lastname3", ZipCode: "10003"},
 	}
 
 	resp, err := dclr().