fix(serverCompress): strip MIME option before checking suffix

Author: marjune163

src/serverCompress/util.go

@@ -17,14 +17,15 @@ func isCompressibleType(contentType string) bool {
 		return true
 	}
 
+	sepIndex := strings.IndexByte(contentType, ';')
+	if sepIndex > 0 {
+		contentType = contentType[:sepIndex]
+	}
+
 	// "image/svg+xml", "application/xhtml+xml", ...
 	if strings.HasSuffix(contentType, "+xml") {
 		return true
 	}
 
-	sepIndex := strings.IndexByte(contentType, ';')
-	if sepIndex > 0 {
-		contentType = contentType[:sepIndex]
-	}
 	return util.Contains(compressibleTypes, contentType)
 }

src/serverCompress/util_test.go

@@ -0,0 +1,32 @@
+package serverCompress
+
+import "testing"
+
+func TestIsCompressibleType(t *testing.T) {
+	if !isCompressibleType("text/plain") {
+		t.Error()
+	}
+	if !isCompressibleType("text/plain; charset=utf-8") {
+		t.Error()
+	}
+
+	if !isCompressibleType("image/svg+xml") {
+		t.Error()
+	}
+	if !isCompressibleType("image/svg+xml; charset=utf-8") {
+		t.Error()
+	}
+	if isCompressibleType("image/png") {
+		t.Error()
+	}
+	if isCompressibleType("image/png; foo=bar") {
+		t.Error()
+	}
+
+	if !isCompressibleType("application/javascript") {
+		t.Error()
+	}
+	if !isCompressibleType("application/javascript; charset=utf-8") {
+		t.Error()
+	}
+}