)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"142c499588ac6c220fc828597c000323dc900001","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add a subclass of general authorization failure for when the response to"},{"line_number":10,"context_line":"the plugin is either invalid JSON or not JSON at all."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3cf2db07a8e76ee17702130e9efb0edf640d293a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAW3%2F%2FiKI%3D","line":11,"updated":"2014-04-13 15:42:04.000000000","message":"this should have a DocImpact\n\nIs there a bug for this?","commit_id":"2b6a3f63201d8f6e6d5cc6bb162336e9fffc1668"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"e3dac45c526dc5ce8ceffa73aeb49ebd6d7c6735","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add a subclass of general authorization failure for when the response to"},{"line_number":10,"context_line":"the plugin is either invalid JSON or not JSON at all."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3cf2db07a8e76ee17702130e9efb0edf640d293a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAW3%2F%2FhhY%3D","line":11,"in_reply_to":"AAAAW3%2F%2FiKI%3D","updated":"2014-04-14 00:02:29.000000000","message":"I agree, I\u0027ll say it again, we\u0027ve gotten a bit lax on bp/bug tracking in the commit messages, we should have it where possible.","commit_id":"2b6a3f63201d8f6e6d5cc6bb162336e9fffc1668"}],"keystoneclient/apiclient/exceptions.py":[{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"e3dac45c526dc5ce8ceffa73aeb49ebd6d7c6735","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"class InvalidAuthResponse(AuthorizationFailure):"},{"line_number":61,"context_line":"    \"\"\"The response from the server is not valid for this auth mechanism.\"\"\""},{"line_number":62,"context_line":""},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"class AuthPluginOptionsMissing(AuthorizationFailure):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FhhM%3D","line":61,"updated":"2014-04-14 00:02:29.000000000","message":"(NIT): This message seems a bit either too verbose or lacking detail.\n\nSuggest \"Invalid response from server for auth mechanism.\" or include the auth mechanism if possible.","commit_id":"2b6a3f63201d8f6e6d5cc6bb162336e9fffc1668"}],"keystoneclient/auth/identity/v2.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"142c499588ac6c220fc828597c000323dc900001","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        try:"},{"line_number":89,"context_line":"            resp_data \u003d resp.json()[\u0027access\u0027]"},{"line_number":90,"context_line":"        except (KeyError, ValueError):"},{"line_number":91,"context_line":"            raise exceptions.InvalidAuthResponse()"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        return access.AccessInfoV2(**resp_data)"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FiJk%3D","line":91,"updated":"2014-04-13 15:42:04.000000000","message":"there should be a docstring updated somewhere that says that this can be returned, otherwise how is anyone going to know how to use it?\n\nI would guess the docs would be on the plugin itself?","commit_id":"2b6a3f63201d8f6e6d5cc6bb162336e9fffc1668"}]}
