plugins/api: use headers.set(), not headers.update()
authorSebastian Spaeth <Sebastian@SSpaeth.de>
Wed, 12 Dec 2012 10:38:51 +0000 (11:38 +0100)
committerSebastian Spaeth <Sebastian@SSpaeth.de>
Fri, 21 Dec 2012 07:12:25 +0000 (08:12 +0100)
The werkzeug.Response().headers do not offer an update() method as
the same key can be twice in the header 'dict'. Thus, iterate over
the header keys and use header.set(key, value) which replaces an
existing header key.

Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
mediagoblin/plugins/api/tools.py

index c4073d23f113265a65c95a5de716228c8d0267d3..333a5682b20913fdcef0ee8b82fba57c014d9af7 100644 (file)
@@ -71,7 +71,7 @@ def json_response(serializable, _disable_cors=False, *args, **kw):
                 'Access-Control-Allow-Origin': '*',
                 'Access-Control-Allow-Methods': 'POST, GET, OPTIONS',
                 'Access-Control-Allow-Headers': 'Content-Type, X-Requested-With'}
-        response.headers.update(cors_headers)
+        (response.headers.set(key, value) for key, value in cors_headers)
 
     return response