)]}'
{"octavia/common/exceptions.py":[{"author":{"_account_id":6579,"name":"Nir Magnezi","email":"nmagnezi@redhat.com","username":"nmagnezi"},"change_message_id":"2b13ce0ec9e6eda5373739ab8dd19a34b36b0105","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    code \u003d 404"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"class NotAuthorized(APIException):"},{"line_number":74,"context_line":"    msg \u003d _(\"Not authorized.\")"},{"line_number":75,"context_line":"    code \u003d 403"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"class InvalidOption(APIException):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f201791_8e428a3d","line":75,"range":{"start_line":73,"start_character":0,"end_line":75,"end_character":14},"updated":"2017-06-25 12:45:35.000000000","message":"Hi Michael,\nBased on this[1], isn\u0027t 403 actually \"Forbidden\"?\n\nLooking at the unit tests, it does seem like this is used by a authenticated user who just does not have permission to do a specific action. So the \u0027403\u0027 looks right, but perhaps we should also change \"Not authorized\" to \"Forbidden\" to reflect that?\n\n\n[1] https://en.wikipedia.org/wiki/HTTP_403#Difference_from_status_.22401_Unauthorized.22","commit_id":"eed291d7679db9c13833401044ea1e4b40c76908"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"2aa32baa7b4e44faa4e24d1dbf88d1618f569a26","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    code \u003d 404"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"class NotAuthorized(APIException):"},{"line_number":74,"context_line":"    msg \u003d _(\"Not authorized.\")"},{"line_number":75,"context_line":"    code \u003d 403"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"class InvalidOption(APIException):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f201791_4ba7a0be","line":75,"range":{"start_line":73,"start_character":0,"end_line":75,"end_character":14},"in_reply_to":"5f201791_8e428a3d","updated":"2017-06-26 21:42:47.000000000","message":"Nir is technically correct here I think. Maybe a more verbose message than \"Forbidden\" would be good though?","commit_id":"eed291d7679db9c13833401044ea1e4b40c76908"}]}