)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18464,"name":"Richard Avelar","email":"csravelar@gmail.com","username":"razone"},"change_message_id":"0384c8412fc73123cd9c8302687b8a23015f6f5f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move protocol to DocumentedRuleDefault"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The overall goal is to define a richer policy for deployers"},{"line_number":10,"context_line":"and operators[0]. To achieve that goal a new policy"},{"line_number":11,"context_line":"class was introduce that requires additional parameters"},{"line_number":12,"context_line":"when defining policy objects."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch switches our  protocol policy object to"},{"line_number":15,"context_line":"the policy.DocumentedRuleDefault and fills the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dfeb2761_90cacf17","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":29},"updated":"2017-04-03 13:06:36.000000000","message":"For consistency, I think we should update this commit as well according to lance\u0027s previous comment: https://review.openstack.org/#/c/449341/3..6//COMMIT_MSG\nOtherwise this is looking good.","commit_id":"4c98b7bff6355e0af0cf80092dcc20ede4aac224"}],"keystone/common/policies/protocol.py":[{"author":{"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"},"change_message_id":"89a05177d9a0e799855d683b5a6342c6006b2fe0","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":19,"context_line":"        name\u003dbase.IDENTITY % \u0027create_protocol\u0027,"},{"line_number":20,"context_line":"        check_str\u003dbase.RULE_ADMIN_REQUIRED,"},{"line_number":21,"context_line":"        description\u003d\u0027Create protocol.\u0027,"},{"line_number":22,"context_line":"        operations\u003d[{\u0027path\u0027: (\u0027/v3/OS-FEDERATION/identity_providers/{idp_id}/\u0027"},{"line_number":23,"context_line":"                              \u0027protocols/{protocol_id}\u0027),"},{"line_number":24,"context_line":"                     \u0027method\u0027: \u0027PUT\u0027}]),"}],"source_content_type":"text/x-python","patch_set":3,"id":"ffe62b97_75de8e3e","line":21,"range":{"start_line":21,"start_character":27,"end_line":21,"end_character":28},"updated":"2017-03-29 16:51:19.000000000","message":"federation","commit_id":"4197f0a02c2c0471088400e9c564f12d9f7d473d"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"5fe791f47c8df5cc97bfc41de353ad0eca6eff9d","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        check_str\u003dbase.RULE_ADMIN_REQUIRED,"},{"line_number":42,"context_line":"        description\u003d\u0027List federated protocols.\u0027,"},{"line_number":43,"context_line":"        operations\u003d[{\u0027path\u0027: (\u0027/v3/OS-FEDERATION/identity_providers/{idp_id}/\u0027"},{"line_number":44,"context_line":"                              \u0027protocols/{protocol_id}\u0027),"},{"line_number":45,"context_line":"                     \u0027method\u0027: \u0027GET\u0027}]),"},{"line_number":46,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":47,"context_line":"        name\u003dbase.IDENTITY % \u0027delete_protocol\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"dfeb2761_4434e04a","line":44,"range":{"start_line":44,"start_character":40,"end_line":44,"end_character":54},"updated":"2017-03-30 17:41:07.000000000","message":"remove this /{protocol_id} as it\u0027s listing.","commit_id":"40d9393f7f480e9a5ab2d2fe6b5f4cbccd5f1e54"}]}