)]}'
{"cinder/policies/messages.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"4e46c3986439db0ef3160a4e268d61cfc42d7783","unresolved":true,"context_lines":[{"line_number":48,"context_line":"                \u0027path\u0027: \u0027/messages\u0027"},{"line_number":49,"context_line":"            }"},{"line_number":50,"context_line":"        ],"},{"line_number":51,"context_line":"        deprecated_rule\u003ddeprecated_get_policy,"},{"line_number":52,"context_line":"    ),"},{"line_number":53,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":54,"context_line":"        name\u003dGET_POLICY,"}],"source_content_type":"text/x-python","patch_set":2,"id":"672ef275_0064799c","line":51,"range":{"start_line":51,"start_character":24,"end_line":51,"end_character":45},"updated":"2021-10-07 18:45:00.000000000","message":"I think we have our wires crossed here. Should this be:\n\n  deprecated_rule\u003ddeprecated_get_all_policy,\n\n?","commit_id":"810733d747968346180cbffcd1698bfe063d7ea1"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"4e46c3986439db0ef3160a4e268d61cfc42d7783","unresolved":true,"context_lines":[{"line_number":60,"context_line":"                \u0027path\u0027: \u0027/messages/{message_id}\u0027"},{"line_number":61,"context_line":"            }"},{"line_number":62,"context_line":"        ],"},{"line_number":63,"context_line":"        deprecated_rule\u003ddeprecated_get_all_policy,"},{"line_number":64,"context_line":"    ),"},{"line_number":65,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":66,"context_line":"        name\u003dDELETE_POLICY,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7589c62f_6429a136","line":63,"range":{"start_line":63,"start_character":24,"end_line":63,"end_character":49},"updated":"2021-10-07 18:45:00.000000000","message":"Should this be:\n\n  deprecated_rule\u003ddeprecated_get_policy,\n\n?","commit_id":"810733d747968346180cbffcd1698bfe063d7ea1"}],"cinder/tests/unit/policies/test_messages.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"636ff12b02f91793816e3052cc6d2ed5bec9df63","unresolved":true,"context_lines":[{"line_number":95,"context_line":"        # in the request matches the one in the context. This is OK because"},{"line_number":96,"context_line":"        # the WSGI controller ensures the context can access the project_id"},{"line_number":97,"context_line":"        # in the request. So in Xena, where scope is not supported, all users"},{"line_number":98,"context_line":"        # will tend to pass the policy check regardless of their project_id."},{"line_number":99,"context_line":"        authorized_users \u003d [user_id]"},{"line_number":100,"context_line":"        unauthorized_users \u003d []"},{"line_number":101,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dc622508_c71464f5","line":98,"updated":"2021-08-30 21:24:14.000000000","message":"Would cinder ever have an interest in collapsing the WSGI logic [0] into the policy check?\n\n[0] https://github.com/openstack/cinder/blob/master/cinder/api/openstack/wsgi.py#L877-L878","commit_id":"d68935df7cad19b17a1d164520091fee4f3302ec"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"8ecccfaca37d4924bbf96481b0f5c4acc3c69e66","unresolved":true,"context_lines":[{"line_number":95,"context_line":"        # in the request matches the one in the context. This is OK because"},{"line_number":96,"context_line":"        # the WSGI controller ensures the context can access the project_id"},{"line_number":97,"context_line":"        # in the request. So in Xena, where scope is not supported, all users"},{"line_number":98,"context_line":"        # will tend to pass the policy check regardless of their project_id."},{"line_number":99,"context_line":"        authorized_users \u003d [user_id]"},{"line_number":100,"context_line":"        unauthorized_users \u003d []"},{"line_number":101,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"8bdc49bf_353d025a","line":98,"in_reply_to":"dc622508_c71464f5","updated":"2021-08-30 22:21:28.000000000","message":"Probably yes, but not in xena :-/","commit_id":"d68935df7cad19b17a1d164520091fee4f3302ec"}]}
