)]}'
{"neutron/policy.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170b06ce89897621e3c40f814215364a99c29dc9","unresolved":true,"context_lines":[{"line_number":208,"context_line":"    match_rule \u003d policy.RuleCheck(\u0027rule\u0027, action)"},{"line_number":209,"context_line":"    registered_rule \u003d _ENFORCER.registered_rules.get(action)"},{"line_number":210,"context_line":"    if registered_rule and registered_rule.scope_types:"},{"line_number":211,"context_line":"        match_rule.scope_types \u003d registered_rule.scope_types"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    resource, enforce_attr_based_check \u003d get_resource_and_action("},{"line_number":214,"context_line":"        action, pluralized)"}],"source_content_type":"text/x-python","patch_set":1,"id":"836624d4_c316d49b","line":211,"updated":"2021-09-10 21:00:56.000000000","message":"I commented in https://review.opendev.org/c/openstack/oslo.policy/+/804980/1/oslo_policy/policy.py#1007\n\nSo setting scope_type in Check bases rule make sense as per neutron use case. I think making scope_type as part of check object will make sense (but this code is all fine in both cases).","commit_id":"48f4e05b6e57846d04b0f575a7768686e7bc8a84"}]}