Skip to content

Commit dae598f

Browse files
committed
Refactor APIHandler usage in httpclient_request.go
1 parent 63d134c commit dae598f

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

httpclient/httpclient_request.go

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -125,10 +125,10 @@ func (c *Client) executeRequestWithRetries(method, endpoint string, body, out in
125125
}()
126126

127127
// Determine which set of encoding and content-type request rules to use
128-
apiHandler := c.APIHandler
128+
// apiHandler := c.APIHandler
129129

130130
// Marshal Request with correct encoding
131-
requestData, err := apiHandler.MarshalRequest(body, method, endpoint, log)
131+
requestData, err := c.APIHandler.MarshalRequest(body, method, endpoint, log)
132132
if err != nil {
133133
return nil, err
134134
}
@@ -418,28 +418,29 @@ func (c *Client) DoMultipartRequest(method, endpoint string, fields map[string]s
418418
}
419419

420420
// Determine which set of encoding and content-type request rules to use
421-
apiHandler := c.APIHandler
421+
//apiHandler := c.APIHandler
422422

423423
// Marshal the multipart form data
424-
requestData, contentType, err := apiHandler.MarshalMultipartRequest(fields, files, log)
424+
requestData, contentType, err := c.APIHandler.MarshalMultipartRequest(fields, files, log)
425425
if err != nil {
426426
return nil, err
427427
}
428428

429429
// Construct URL using the ConstructAPIResourceEndpoint function
430-
url := apiHandler.ConstructAPIResourceEndpoint(c.InstanceName, endpoint, log)
430+
url := c.APIHandler.ConstructAPIResourceEndpoint(c.InstanceName, endpoint, log)
431431

432432
// Create the request
433433
req, err := http.NewRequest(method, url, bytes.NewBuffer(requestData))
434434
if err != nil {
435435
return nil, err
436436
}
437437

438-
// Get Request Headers dynamically based on api handler
439-
acceptHeader := apiHandler.GetAcceptHeader()
438+
// Initialize HeaderManager
439+
headerManager := NewHeaderManager(req, log, c.APIHandler, c.Token)
440440

441-
// Set Request Headers
442-
c.SetRequestHeaders(req, contentType, acceptHeader, log)
441+
// Use HeaderManager to set headers
442+
headerManager.SetContentType(contentType) // Content-Type from MarshalMultipartRequest
443+
headerManager.SetRequestHeaders(endpoint) // Set other standard headers
443444

444445
// Execute the request
445446
resp, err := c.executeHTTPRequest(req, log, method, endpoint)

0 commit comments

Comments
 (0)