)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"e0dbdb028e972c96a19136556be23e99072bf888","unresolved":false,"context_lines":[{"line_number":18,"context_line":"In addition to not using admin_token on principle, the"},{"line_number":19,"context_line":"auth_token middleware exhibits odd behavior if the identity"},{"line_number":20,"context_line":"server responds with a 401 -- it switches to using the admin"},{"line_number":21,"context_line":"user and password."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In preparing for removal of the admin_token option, the option"},{"line_number":24,"context_line":"is now deprecated. A warning will be logged if it\u0027s set."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAXH%2F%2F1RU%3D","line":21,"updated":"2014-04-17 04:53:16.000000000","message":"the latter part of this paragraph in the commit message doesn\u0027t relate back to the fix. The fix just mentions deprecation, and suggests using username/password. Not that it falls back to it. Suggest remove this part, and the related bug at the bottom?","commit_id":"869a9175c5c19d02879cea05e7b5bb7023189562"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"adbae1f2be70db53c3785301ada310c102f51f38","unresolved":false,"context_lines":[{"line_number":18,"context_line":"In addition to not using admin_token on principle, the"},{"line_number":19,"context_line":"auth_token middleware exhibits odd behavior if the identity"},{"line_number":20,"context_line":"server responds with a 401 -- it switches to using the admin"},{"line_number":21,"context_line":"user and password."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In preparing for removal of the admin_token option, the option"},{"line_number":24,"context_line":"is now deprecated. A warning will be logged if it\u0027s set."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAXH%2F%2Fw44%3D","line":21,"in_reply_to":"AAAAXH%2F%2F1RU%3D","updated":"2014-04-17 16:09:31.000000000","message":"Done -- I was asked to add the Related-Bug by Morgan in patch set 1, now I\u0027m removing it... I\u0027m guessing I\u0027ll be asked to add it again later...","commit_id":"869a9175c5c19d02879cea05e7b5bb7023189562"}],"keystoneclient/middleware/auth_token.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"f156b709e0e32811c2c0f63652c3cbc96d1d6ea5","unresolved":false,"context_lines":[{"line_number":242,"context_line":"               secret\u003dTrue,"},{"line_number":243,"context_line":"               help\u003d\u0027Single shared secret with the Keystone configuration\u0027"},{"line_number":244,"context_line":"               \u0027 used for bootstrapping a Keystone installation, or otherwise\u0027"},{"line_number":245,"context_line":"               \u0027 bypassing the normal authentication process.\u0027),"},{"line_number":246,"context_line":"    cfg.StrOpt(\u0027admin_user\u0027,"},{"line_number":247,"context_line":"               help\u003d\u0027Keystone account username\u0027),"},{"line_number":248,"context_line":"    cfg.StrOpt(\u0027admin_password\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2Fbd0%3D","side":"PARENT","line":245,"updated":"2014-04-14 16:30:24.000000000","message":"the old help message is a lot more useful than \"Administrative token\" which is just redundant with the option name itself. please restore!","commit_id":"8bc8275bf3b4b1fd644102963f2980d23b1bffc9"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"103473daf1dce4445a3fc13779d0a5aa4c2a6041","unresolved":false,"context_lines":[{"line_number":242,"context_line":"               secret\u003dTrue,"},{"line_number":243,"context_line":"               help\u003d\u0027Single shared secret with the Keystone configuration\u0027"},{"line_number":244,"context_line":"               \u0027 used for bootstrapping a Keystone installation, or otherwise\u0027"},{"line_number":245,"context_line":"               \u0027 bypassing the normal authentication process.\u0027),"},{"line_number":246,"context_line":"    cfg.StrOpt(\u0027admin_user\u0027,"},{"line_number":247,"context_line":"               help\u003d\u0027Keystone account username\u0027),"},{"line_number":248,"context_line":"    cfg.StrOpt(\u0027admin_password\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2FSB4%3D","side":"PARENT","line":245,"in_reply_to":"AAAAW3%2F%2Fbd0%3D","updated":"2014-04-15 15:35:10.000000000","message":"Done","commit_id":"8bc8275bf3b4b1fd644102963f2980d23b1bffc9"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"f156b709e0e32811c2c0f63652c3cbc96d1d6ea5","unresolved":false,"context_lines":[{"line_number":243,"context_line":"               help\u003d\u0027Administative token. This option should not be used, use\u0027"},{"line_number":244,"context_line":"               \u0027 `admin_user` and `admin_password` instead.\u0027"},{"line_number":245,"context_line":"               \u0027 This option is deprecated in Juno and may be removed in the\u0027"},{"line_number":246,"context_line":"               \u0027 K release.\u0027),"},{"line_number":247,"context_line":"    cfg.StrOpt(\u0027admin_user\u0027,"},{"line_number":248,"context_line":"               help\u003d\u0027Keystone account username\u0027),"},{"line_number":249,"context_line":"    cfg.StrOpt(\u0027admin_password\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2FbeE%3D","line":246,"updated":"2014-04-14 16:30:24.000000000","message":"the client doesn\u0027t have named releases, so this doesn\u0027t really make sense in this context","commit_id":"af61b5cfbfbeecb617bede38132d383deb241f4d"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"103473daf1dce4445a3fc13779d0a5aa4c2a6041","unresolved":false,"context_lines":[{"line_number":243,"context_line":"               help\u003d\u0027Administative token. This option should not be used, use\u0027"},{"line_number":244,"context_line":"               \u0027 `admin_user` and `admin_password` instead.\u0027"},{"line_number":245,"context_line":"               \u0027 This option is deprecated in Juno and may be removed in the\u0027"},{"line_number":246,"context_line":"               \u0027 K release.\u0027),"},{"line_number":247,"context_line":"    cfg.StrOpt(\u0027admin_user\u0027,"},{"line_number":248,"context_line":"               help\u003d\u0027Keystone account username\u0027),"},{"line_number":249,"context_line":"    cfg.StrOpt(\u0027admin_password\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2FSB0%3D","line":246,"in_reply_to":"AAAAW3%2F%2FbeE%3D","updated":"2014-04-15 15:35:10.000000000","message":"Done","commit_id":"af61b5cfbfbeecb617bede38132d383deb241f4d"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"630254d78e814376cf1d0a3aa448a283a76352a9","unresolved":false,"context_lines":[{"line_number":245,"context_line":"               \u0027 This option is deprecated and may be removed in a future\u0027"},{"line_number":246,"context_line":"               \u0027 release. Single shared secret with the Keystone configuration\u0027"},{"line_number":247,"context_line":"               \u0027 used for bootstrapping a Keystone installation, or otherwise\u0027"},{"line_number":248,"context_line":"               \u0027 bypassing the normal authentication process.\u0027),"},{"line_number":249,"context_line":"    cfg.StrOpt(\u0027admin_user\u0027,"},{"line_number":250,"context_line":"               help\u003d\u0027Keystone account username\u0027),"},{"line_number":251,"context_line":"    cfg.StrOpt(\u0027admin_password\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2Fw2o%3D","line":248,"updated":"2014-04-17 16:12:36.000000000","message":"I\u0027d prefer the first sentence \"This option should not...\" be moved to the end, as the whole paragraph would flow a bit better","commit_id":"68d5094dd9a41d4adef02c9f3a118005d0e9101b"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"b02b0261fde4d3b94216049e618315a450667120","unresolved":false,"context_lines":[{"line_number":245,"context_line":"               \u0027 This option is deprecated and may be removed in a future\u0027"},{"line_number":246,"context_line":"               \u0027 release. Single shared secret with the Keystone configuration\u0027"},{"line_number":247,"context_line":"               \u0027 used for bootstrapping a Keystone installation, or otherwise\u0027"},{"line_number":248,"context_line":"               \u0027 bypassing the normal authentication process.\u0027),"},{"line_number":249,"context_line":"    cfg.StrOpt(\u0027admin_user\u0027,"},{"line_number":250,"context_line":"               help\u003d\u0027Keystone account username\u0027),"},{"line_number":251,"context_line":"    cfg.StrOpt(\u0027admin_password\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2Fwy8%3D","line":248,"in_reply_to":"AAAAXH%2F%2Fw2o%3D","updated":"2014-04-17 16:21:01.000000000","message":"Done","commit_id":"68d5094dd9a41d4adef02c9f3a118005d0e9101b"}]}
