)]}'
{"swift/common/middleware/bulk.py":[{"author":{"_account_id":1009,"name":"Greg Lange","email":"greglange@gmail.com","username":"greglange"},"change_message_id":"7fbe0b1a1b31fa77897afaacbc872c58daf71fb4","unresolved":false,"context_lines":[{"line_number":323,"context_line":"                raise HTTPOk(request\u003dreq)"},{"line_number":324,"context_line":"            if failed_files:"},{"line_number":325,"context_line":"                raise failed_file_response_type(request\u003dreq)"},{"line_number":326,"context_line":"            raise HTTPBadRequest(\u0027Invalid bulk delete.\u0027)"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"        except HTTPException, err:"},{"line_number":329,"context_line":"            resp_dict \u003d {\u0027Response Status\u0027: err.status,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2FlRU%3D","line":326,"updated":"2013-05-02 22:41:43.000000000","message":"This flow control is a little weird.  Could you at least put a note in about it?  In real life you described your reasons for this but it is pretty non-obvious what is going on here.","commit_id":"3b74eaa6077510e5edf9e1b66ab2453ccb898fe5"}],"swift/common/middleware/slo.py":[{"author":{"_account_id":1009,"name":"Greg Lange","email":"greglange@gmail.com","username":"greglange"},"change_message_id":"7fbe0b1a1b31fa77897afaacbc872c58daf71fb4","unresolved":false,"context_lines":[{"line_number":311,"context_line":"            raise HTTPPreconditionFailed("},{"line_number":312,"context_line":"                request\u003dreq, body\u003d\u0027Invalid UTF8 or contains NULL\u0027)"},{"line_number":313,"context_line":"        try:"},{"line_number":314,"context_line":"            vrs, account, container, obj \u003d req.split_path(1, 4, True)"},{"line_number":315,"context_line":"        except ValueError:"},{"line_number":316,"context_line":"            raise HTTPBadRequest(\u0027Not an SLO manifest\u0027)"},{"line_number":317,"context_line":"        new_env \u003d req.environ.copy()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2FlRc%3D","line":314,"updated":"2013-05-02 22:41:43.000000000","message":"Since you are expecting 4 segments in the path and not checking for None in the values returned past the first value, I think the minsegs in the call to split_path() should be 4 instead of 1.","commit_id":"3b74eaa6077510e5edf9e1b66ab2453ccb898fe5"}]}