use a default content-type if only media parameters are present (#140)

jhillyerd · Dec 23, 2019 · 3f81043 · 3f81043
1 parent f9d5121
commit 3f81043
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/header.go b/header.go
@@ -277,6 +277,11 @@ func fixMangledMediaType(mtype, sep string) string {
 	}
 	parts := strings.Split(mtype, sep)
 	mtype = ""
+	if strings.Contains(parts[0], "=") {
+		// A parameter pair at this position indicates we are missing a content-type.
+		parts[0] = fmt.Sprintf("%s%s %s", ctAppOctetStream, sep, parts[0])
+		parts = strings.Split(strings.Join(parts, sep), sep)
+	}
 	for i, p := range parts {
 		switch i {
 		case 0:

diff --git a/header_test.go b/header_test.go
@@ -214,6 +214,11 @@ func TestFixMangledMediaType(t *testing.T) {
 			sep:   ";",
 			want:  ctPlaceholder + "; name=\"file.two\"",
 		},
+		{
+			input: "charset=binary; name=\"logoleft.jpg\"",
+			sep:   ";",
+			want:  "application/octet-stream; charset=binary; name=\"logoleft.jpg\"",
+		},
 		{
 			input: "one/two;iso-8859-1",
 			sep:   ";",