)]}'
{"swift/common/middleware/tempurl.py":[{"author":{"_account_id":13852,"name":"Romain LE DISEZ","email":"romain.le-disez@corp.ovh.com","username":"rledisez"},"change_message_id":"f416742ca3b92af6e4e5eef96172ed7dced677ae","unresolved":false,"context_lines":[{"line_number":388,"context_line":"        self.outgoing_allow_headers_startswith \u003d \\"},{"line_number":389,"context_line":"            [h[:-1] for h in headers if h.endswith(\u0027*\u0027)]"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        query_params \u003d [p"},{"line_number":392,"context_line":"                        for p in conf.get(\u0027allowed_query_params\u0027,"},{"line_number":393,"context_line":"                                          \u0027\u0027).split()]"},{"line_number":394,"context_line":"        #: Query params which are conserved through the pipeline."}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_4b1d97bb","line":391,"updated":"2017-06-01 09:20:07.000000000","message":"I know all other params, are space separated for this middleware, but it seems weird as everything else in swift is comma separated (list_from_csv). Should we use comma separated for this param?","commit_id":"b4cb7a4baad7492f472c952628ddad28e3d29e03"},{"author":{"_account_id":20764,"name":"Sachin","email":"psachin@redhat.com","username":"psachin"},"change_message_id":"74fef81f9aeb65222ad1d6093603c9beb2907539","unresolved":false,"context_lines":[{"line_number":388,"context_line":"        self.outgoing_allow_headers_startswith \u003d \\"},{"line_number":389,"context_line":"            [h[:-1] for h in headers if h.endswith(\u0027*\u0027)]"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        query_params \u003d [p"},{"line_number":392,"context_line":"                        for p in conf.get(\u0027allowed_query_params\u0027,"},{"line_number":393,"context_line":"                                          \u0027\u0027).split()]"},{"line_number":394,"context_line":"        #: Query params which are conserved through the pipeline."}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_2b64a321","line":391,"in_reply_to":"df140735_4b1d97bb","updated":"2017-06-01 09:32:52.000000000","message":"I agree, comma separated delimiter will avoid typos/mistakes as they are more prominent to eyes than spaces. But..here Christopher is trying to stay consistent with other params. User may not like to have mix of comma as well as space separated delimiter.","commit_id":"b4cb7a4baad7492f472c952628ddad28e3d29e03"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"45bf42b420bb01b90d83a1881f7083e07d8917a9","unresolved":false,"context_lines":[{"line_number":865,"context_line":"        \u0027incoming_allow_headers\u0027: DEFAULT_INCOMING_ALLOW_HEADERS,"},{"line_number":866,"context_line":"        \u0027outgoing_remove_headers\u0027: DEFAULT_OUTGOING_REMOVE_HEADERS,"},{"line_number":867,"context_line":"        \u0027outgoing_allow_headers\u0027: DEFAULT_OUTGOING_ALLOW_HEADERS,"},{"line_number":868,"context_line":"        \u0027allowed_digests\u0027: DEFAULT_ALLOWED_DIGESTS,"},{"line_number":869,"context_line":"    }"},{"line_number":870,"context_line":"    info_conf \u003d {k: conf.get(k, v).split() for k, v in defaults.items()}"},{"line_number":871,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9fdfeff1_6f1fe46b","line":868,"updated":"2019-02-14 21:10:48.000000000","message":"Should we expose the new config in /info?","commit_id":"7877ee55c100d80025052a3cc6e2d2802343cd23"}]}