fix(gzhttp): preserve qvalue when extra parameters follow in Accept-Encoding (#1116)

parseCoding() reset qvalue to DefaultQValue on every loop iteration,
causing "gzip;q=0.5;level=6" to incorrectly return qvalue=1.0 instead
of 0.5.

Move qvalue initialization before the loop.

Signed-off-by: Mathias Bogaert <mathias.bogaert@gmail.com>

Author: analytically

gzhttp/compress.go

@@ -881,9 +881,9 @@ func parseCoding(s string) (coding string, qvalue float64, err error) {
 		}
 		return coding, DefaultQValue, err
 	}
+	qvalue = DefaultQValue
 	for n, part := range strings.Split(s, ";") {
 		part = strings.TrimSpace(part)
-		qvalue = DefaultQValue
 
 		if n == 0 {
 			coding = strings.ToLower(part)

gzhttp/compress_test.go

@@ -40,6 +40,9 @@ func TestParseEncodings(t *testing.T) {
 		// More random stuff
 		"AAA;q=1":     {"aaa": 1.0},
 		"BBB ; q = 2": {"bbb": 1.0},
+
+		"gzip;q=0.5;level=6":     {"gzip": 0.5},
+		"gzip;q=0.3;level=6;x=y": {"gzip": 0.3},
 	}
 
 	for eg, exp := range examples {