removed some functions which literally just wrapped another function and nothing else - 2 of them!

Author: thejoeker12

response/error.go

@@ -57,7 +57,7 @@ func HandleAPIErrorResponse(resp *http.Response, sugar *zap.SugaredLogger) *APIE
 		return apiError
 	}
 
-	mimeType, _ := ParseContentTypeHeader(resp.Header.Get("Content-Type"))
+	mimeType, _ := parseHeader(resp.Header.Get("Content-Type"))
 	switch mimeType {
 	case "application/json":
 		parseJSONResponse(bodyBytes, apiError)

response/parse.go

@@ -3,16 +3,6 @@ package response
 
 import "strings"
 
-// ParseContentTypeHeader parses the Content-Type header and returns the MIME type and any parameters.
-func ParseContentTypeHeader(header string) (string, map[string]string) {
-	return parseHeader(header)
-}
-
-// ParseContentDisposition parses the Content-Disposition header and returns the type and any parameters.
-func ParseContentDisposition(header string) (string, map[string]string) {
-	return parseHeader(header)
-}
-
 // parseHeader generalizes the parsing of headers like Content-Type and Content-Disposition.
 // It extracts the main value (e.g., MIME type for Content-Type) and any parameters (like charset).
 func parseHeader(header string) (string, map[string]string) {

response/status.go response/statuscode_helpers.goresponse/status.go renamed to response/statuscode_helpers.go

@@ -46,7 +46,7 @@ func HandleAPISuccessResponse(resp *http.Response, out interface{}, sugar *zap.S
 	// Use the buffer to create a new io.Reader for unmarshalling
 	bodyReader := bytes.NewReader(bodyBytes)
 
-	mimeType, _ := ParseContentTypeHeader(resp.Header.Get("Content-Type"))
+	mimeType, _ := parseHeader(resp.Header.Get("Content-Type"))
 	contentDisposition := resp.Header.Get("Content-Disposition")
 
 	if handler, ok := responseUnmarshallers[mimeType]; ok {
@@ -123,7 +123,7 @@ func handleBinaryData(reader io.Reader, sugar *zap.SugaredLogger, out interface{
 
 	// Handle Content-Disposition if present
 	if contentDisposition != "" {
-		_, params := ParseContentDisposition(contentDisposition)
+		_, params := parseHeader(contentDisposition)
 		if filename, ok := params["filename"]; ok {
 			sugar.Debug("Extracted filename from Content-Disposition", zap.String("filename", filename))
 			// Additional processing for the filename can be done here if needed

response/success.go

@@ -35,7 +35,7 @@ func TestParseContentTypeHeader(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			gotType, gotParams := ParseContentTypeHeader(tt.header)
+			gotType, gotParams := parseHeader(tt.header)
 			if gotType != tt.wantType {
 				t.Errorf("ParseContentTypeHeader() gotType = %v, want %v", gotType, tt.wantType)
 			}
@@ -70,7 +70,7 @@ func TestParseContentDisposition(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			gotType, gotParams := ParseContentDisposition(tt.header)
+			gotType, gotParams := parseHeader(tt.header)
 			if gotType != tt.wantType {
 				t.Errorf("ParseContentDisposition() gotType = %v, want %v", gotType, tt.wantType)
 			}