Refactor multipart request handling and logging

ShocOne · ShocOne · commit 974d55690f75 · 2024-06-05T17:10:44.000+01:00
diff --git a/apiintegrations/jamfpro/jamfpro_api_headers.go b/apiintegrations/jamfpro/jamfpro_api_headers.go
@@ -32,8 +32,8 @@ func (j *JamfAPIHandler) GetContentTypeHeader(endpoint string, log logger.Logger
 
 	// Special case for package upload endpoint
 	if strings.HasPrefix(endpoint, "/api/v1/packages") && strings.HasSuffix(endpoint, "/upload") {
-		j.Logger.Debug("Content-Type for package upload endpoint defaulting to multipart/form-data", zap.String("endpoint", endpoint))
-		return "multipart/form-data"
+		j.Logger.Debug("Skipping Content-Type setting for package upload endpoint. Multipart request will set", zap.String("endpoint", endpoint))
+		return "" // Skip setting Content-Type here
 	}
 
 	// If no specific configuration is found, then check for standard URL patterns.

Original file line number	Diff line number	Diff line change
`@@ -32,8 +32,8 @@ func (j *JamfAPIHandler) GetContentTypeHeader(endpoint string, log logger.Logger`
`32`	`32`
`33`	`33`	`// Special case for package upload endpoint`
`34`	`34`	`if strings.HasPrefix(endpoint, "/api/v1/packages") && strings.HasSuffix(endpoint, "/upload") {`
`35`		`- j.Logger.Debug("Content-Type for package upload endpoint defaulting to multipart/form-data", zap.String("endpoint", endpoint))`
`36`		`- return "multipart/form-data"`
	`35`	`+ j.Logger.Debug("Skipping Content-Type setting for package upload endpoint. Multipart request will set", zap.String("endpoint", endpoint))`
	`36`	`+ return "" // Skip setting Content-Type here`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`// If no specific configuration is found, then check for standard URL patterns.`