)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"b4bb998b2f195433b50b463230e77a06109c1158","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Marek Denis \u003cmarek.denis@cern.ch\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-03-06 10:58:24 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix docstrings in federation related modules."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The docstrings in keystone/auth/plugins/saml2.py and"},{"line_number":10,"context_line":"keystone/contrib/federation/routers.py should be updated so they"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAVn%2F%2FV2c%3D","line":7,"updated":"2014-03-06 18:25:55.000000000","message":"The first line of a commit message shouldn\u0027t end in a period","commit_id":"f85379d7aef60674a8042b02b19a94b6710a7eed"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"b4bb998b2f195433b50b463230e77a06109c1158","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix docstrings in federation related modules."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The docstrings in keystone/auth/plugins/saml2.py and"},{"line_number":10,"context_line":"keystone/contrib/federation/routers.py should be updated so they"},{"line_number":11,"context_line":"properly describe the code."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAVn%2F%2FV1k%3D","line":9,"updated":"2014-03-06 18:25:55.000000000","message":"This should be rephrased as a statement of what was done. This sounds more like a TODO.","commit_id":"f85379d7aef60674a8042b02b19a94b6710a7eed"}],"keystone/auth/plugins/saml2.py":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"f82dbb0819fef626a89a1ccd1fb1bab6d6fb4b91","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        In addition to ``user_id`` in ``auth_context``, the ``saml2``"},{"line_number":45,"context_line":"        plugin sets ``group_ids``. Also, when handling unscoped token"},{"line_number":46,"context_line":"        the ``OS-FEDERATION:identity_provider`` and ``OS-FEDERATION:protocol``"},{"line_number":47,"context_line":"        are set."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAVn%2F%2FjLg%3D","line":46,"updated":"2014-03-05 13:30:13.000000000","message":"I can\u0027t put my finger on it, but this sentence still sounds a little clunky. I think the may be the leading \u0027Also\u0027.","commit_id":"790db7f43a4eda5559f828de30203bc057208146"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"d7a8593aef033424275d0da76cf7270925dda6f9","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        In addition to ``user_id`` in ``auth_context``, the ``saml2``"},{"line_number":45,"context_line":"        plugin sets ``group_ids``. Also, when handling unscoped token"},{"line_number":46,"context_line":"        the ``OS-FEDERATION:identity_provider`` and ``OS-FEDERATION:protocol``"},{"line_number":47,"context_line":"        are set."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAVn%2F%2Fh2c%3D","line":46,"in_reply_to":"AAAAVn%2F%2FjEg%3D","updated":"2014-03-05 15:41:41.000000000","message":"Done","commit_id":"790db7f43a4eda5559f828de30203bc057208146"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"0156c4724fec641fdb4bba49e13d3834611ca404","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        In addition to ``user_id`` in ``auth_context``, the ``saml2``"},{"line_number":45,"context_line":"        plugin sets ``group_ids``. Also, when handling unscoped token"},{"line_number":46,"context_line":"        the ``OS-FEDERATION:identity_provider`` and ``OS-FEDERATION:protocol``"},{"line_number":47,"context_line":"        are set."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAVn%2F%2FjEg%3D","line":46,"in_reply_to":"AAAAVn%2F%2FjLg%3D","updated":"2014-03-05 13:45:16.000000000","message":"I\u0027d drop the \"Also,\" make \"token\" plural, and add a comma after \"tokens\", and drop \"the\" in the second part of the sentence:\n\n  When handling unscoped tokens, ``OS-FEDERATION:identity_provider`` and ``OS-FEDERATION:protocol`` are set.","commit_id":"790db7f43a4eda5559f828de30203bc057208146"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"03b829f839e19bf6584d244c5dc0be92d62670a1","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        In addition to ``user_id`` in ``auth_context``, the ``saml2``"},{"line_number":45,"context_line":"        plugin sets ``group_ids``. When handling unscoped tokens,"},{"line_number":46,"context_line":"        ``OS-FEDERATION:identity_provider`` and ``OS-FEDERATION:protocol``"},{"line_number":47,"context_line":"        are set."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        \"\"\""},{"line_number":50,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FejA%3D","line":47,"updated":"2014-03-05 21:13:59.000000000","message":"add \"as well\" after set","commit_id":"2caf63a0cbafbdd56fb9014a0ec6b299ee3f0c99"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"b799e770f74c68df2f2afd107fc490516ea59e68","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        In addition to ``user_id`` in ``auth_context``, the ``saml2``"},{"line_number":45,"context_line":"        plugin sets ``group_ids``. When handling unscoped tokens,"},{"line_number":46,"context_line":"        ``OS-FEDERATION:identity_provider`` and ``OS-FEDERATION:protocol``"},{"line_number":47,"context_line":"        are set."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        \"\"\""},{"line_number":50,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FZ0A%3D","line":47,"in_reply_to":"AAAAVn%2F%2FejA%3D","updated":"2014-03-06 09:58:11.000000000","message":"added.","commit_id":"2caf63a0cbafbdd56fb9014a0ec6b299ee3f0c99"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"51d652f12a946e5e5f28fb169a84777d2bcb026f","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        In addition to ``user_id`` in ``auth_context``, the ``saml2``"},{"line_number":45,"context_line":"        plugin sets ``group_ids``. When handling unscoped tokens,"},{"line_number":46,"context_line":"        ``OS-FEDERATION:identity_provider`` and ``OS-FEDERATION:protocol``"},{"line_number":47,"context_line":"        are set as well."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        \"\"\""},{"line_number":50,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVn%2F%2FTg8%3D","line":47,"updated":"2014-03-06 23:02:06.000000000","message":"(NIT): suggest remove \"as well\". It is superfluous.","commit_id":"23b8592137ee64fda502df69bf6268e8ea2f143a"}]}