)]}'
{"swift/common/middleware/tempurl.py":[{"author":{"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"},"change_message_id":"43d8600f9dc68a1347ca9ceb81217c5498f1ebbd","unresolved":false,"context_lines":[{"line_number":253,"context_line":"            A whitespace delimited list of request methods that are"},{"line_number":254,"context_line":"            allowed to be used with a temporary URL."},{"line_number":255,"context_line":"            Default: \u0027GET HEAD PUT POST DELETE\u0027"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"    The proxy logs created for any subrequests made will have swift.source set"},{"line_number":258,"context_line":"    to \"TU\"."},{"line_number":259,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a29b11f_a7bd8203","line":256,"updated":"2015-10-22 13:22:40.000000000","message":"Docstring says \"whitespace delimited list of ...\", but in filter_factory func it initializes to default as already splitted list of strings. It is quite confusing.","commit_id":"41da2edc06122bc5ea6d0b02f5cd2e73d877cc54"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"a67eba11c4dc61d1a9942bc602d9ef5fca0fd491","unresolved":false,"context_lines":[{"line_number":253,"context_line":"            A whitespace delimited list of request methods that are"},{"line_number":254,"context_line":"            allowed to be used with a temporary URL."},{"line_number":255,"context_line":"            Default: \u0027GET HEAD PUT POST DELETE\u0027"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"    The proxy logs created for any subrequests made will have swift.source set"},{"line_number":258,"context_line":"    to \"TU\"."},{"line_number":259,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a26ad4f_4915cc3d","line":256,"in_reply_to":"3a29b11f_a7bd8203","updated":"2015-11-29 23:32:50.000000000","message":"The docstring is describing the configuration settings as they appear in proxy-server.conf, not the structure of the conf parameter. Will move all of this up to the module-level docstring.","commit_id":"41da2edc06122bc5ea6d0b02f5cd2e73d877cc54"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"5d2512f7f2f153ee7a47e53ce5f29bf3e1724196","unresolved":false,"context_lines":[{"line_number":117,"context_line":"---------------------"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"This middleware understands the following configuration settings::"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"    incoming_remove_headers"},{"line_number":122,"context_line":"        The headers to remove from incoming requests. Simply a"},{"line_number":123,"context_line":"        whitespace delimited list of header names and names can"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a8ffd7b_0e66b529","line":120,"updated":"2015-11-30 09:33:56.000000000","message":"I think this block could be improved a little bit; it\u0027s now only an intended code-block (http://docs-draft.openstack.org/12/232712/2/check/gate-swift-docs/79a8f9b//doc/build/html/middleware.html#cluster-configuration).\n\nHow about removing the double colon, removing one level of intendation and thus making a definition list of it?\n\n http://docutils.sourceforge.net/docs/ref/rst/restructuredtext.html#definition-lists\n\nI think that would improve this even more.","commit_id":"9f96552bc9a6caf880cdad0c6c809590444c5d51"}]}