Merge pull request #65 from shogo82148/yaml-is-not-binary

Add support for YAML media type in isBinary function
shogo82148 · Feb 5, 2024 · 60457f1 · 60457f1
2 parents 5c90f74 + 2da4203
commit 60457f1
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/ridgenative.go b/ridgenative.go
@@ -378,6 +378,9 @@ func isBinary(contentType string) bool {
 	if strings.EqualFold(mediaType, "application/json") {
 		return false
 	}
+	if strings.EqualFold(mediaType, "application/yaml") {
+		return false
+	}
 	if strings.EqualFold(mediaType, "application/javascript") {
 		return false
 	}
@@ -393,6 +396,9 @@ func isBinary(contentType string) bool {
 	if strings.EqualFold(suffix, "+json") {
 		return false
 	}
+	if strings.EqualFold(suffix, "+yaml") {
+		return false
+	}
 	if strings.EqualFold(suffix, "+xml") {
 		return false
 	}

diff --git a/ridgenative_test.go b/ridgenative_test.go
@@ -960,10 +960,12 @@ func TestIsBinary(t *testing.T) {
 		{"text/xml", false},
 		{"application/json", false},
 		{"application/javascript", false},
+		{"application/yaml", false},
 		{"application/xml", false},
-		{"application/foo+json", false},
-		{"application/foo+xml", false},
-		{"application/foo+xml ; charset=utf8", false},
+		{"application/vnd.foo+json", false},
+		{"application/vnd.foo+yaml", false},
+		{"application/vnd.foo+xml", false},
+		{"application/vnd.foo+xml ; charset=utf8", false},
 		{"application/octet-stream", true},
 		{"image/jpeg", true},
 		{"audio/mpeg", true},