Handle non-numeric q value gracefully

Author: scop

mimeparse.py

@@ -72,7 +72,10 @@ def parse_media_range(range):
     necessary.
     """
     (type, subtype, params) = parse_mime_type(range)
-    if not params.get('q') or not 0 <= float(params['q']) <= 1:
+    try:
+        if not params.get('q') or not 0 <= float(params['q']) <= 1:
+            params['q'] = '1'
+    except ValueError:  # from float()
         params['q'] = '1'
 
     return (type, subtype, params)

testdata.json

@@ -8,6 +8,7 @@
     ["application/xml ; q=1;b=other",["application", "xml", {"q": "1", "b":"other"}]],
     ["application/xml ; q=2;b=other",["application", "xml", {"q": "1", "b":"other"}]],
     ["application/xml ; q=0",["application", "xml", {"q": "0"}]],
+    ["application/xml ; q=foo", ["application", "xml", {"q": "1"}]],
     [" *; q=.2",["*", "*", {"q": ".2"}]]
 ],