)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"9e09e8373b6adb8fdcf7fe586ca277cb78f26e08","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Special 401/403 debug responses with qa disabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Special change for cases of responses extracted to variable,"},{"line_number":10,"context_line":"blocked by test hacks, hence hidden by \"noqa\" comment."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3d27fa6a8ff2a2699d352a9905bce20b731fe50c"},{"line_number":13,"context_line":"Closes-Bug: 1625120"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a98d1f4_c8e1eb2b","line":10,"updated":"2016-10-03 17:28:15.000000000","message":"I\u0027m having a hard time following this commit message. I understand what we\u0027re fixing but the how is a little hard to understand given the wording.","commit_id":"ff1ba979cf175887e62b7575283d135ff0a8ca69"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"a3785e5cf6e1b8c4f4a063f621e9e281cb425811","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Special 401/403 debug responses with qa disabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Special change for cases of responses extracted to variable,"},{"line_number":10,"context_line":"blocked by test hacks, hence hidden by \"noqa\" comment."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3d27fa6a8ff2a2699d352a9905bce20b731fe50c"},{"line_number":13,"context_line":"Related-Bug: 1625120"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba5201f7_d3edb08c","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":54},"updated":"2017-01-03 02:17:15.000000000","message":"I am not following what is the issue and what we\u0027re fixing here (also pointed out by Lance in a previous patchset).\n\nLooking at the changed code, I don\u0027t see any response extracted to a variable.","commit_id":"dfeda349d567364a569ce9ca9a3b4df388052560"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"a3785e5cf6e1b8c4f4a063f621e9e281cb425811","unresolved":false,"context_lines":[{"line_number":10,"context_line":"blocked by test hacks, hence hidden by \"noqa\" comment."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3d27fa6a8ff2a2699d352a9905bce20b731fe50c"},{"line_number":13,"context_line":"Related-Bug: 1625120"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba5201f7_13001834","line":13,"updated":"2017-01-03 02:17:15.000000000","message":"I do not see how this is related to that bug","commit_id":"dfeda349d567364a569ce9ca9a3b4df388052560"}],"keystone/common/utils.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"9e09e8373b6adb8fdcf7fe586ca277cb78f26e08","unresolved":false,"context_lines":[{"line_number":545,"context_line":"        return auth_context[\u0027token\u0027]"},{"line_number":546,"context_line":"    except KeyError:"},{"line_number":547,"context_line":"        msg \u003d _LW(\"Couldn\u0027t find the auth context.\")"},{"line_number":548,"context_line":"        LOG.warning(msg)  # noqa"},{"line_number":549,"context_line":"        raise exception.Unauthorized(msg)"},{"line_number":550,"context_line":""},{"line_number":551,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3a98d1f4_e86067a7","line":548,"updated":"2016-10-03 17:28:15.000000000","message":"Can we enforce this through tests? If other people rely on specific information presented through the logs (testing or otherwise), then we need to make sure we\u0027re not regressing on accident. That would be easier to do with some form of test.","commit_id":"ff1ba979cf175887e62b7575283d135ff0a8ca69"}]}
