From c9148db371dfa449830d2fbe7c1345135ebddf3f Mon Sep 17 00:00:00 2001 From: Vishakha Agarwal Date: Thu, 19 Sep 2019 18:45:30 +0530 Subject: [PATCH] Implement scope type checking for Project Endpoints This change updates the Project Endpoints policies to understand the scope types for Project Endpoints. This adds the test cases too. Change-Id: Id18036325b2f5b8836076408ecdd64523b19cbba Closes-Bug: #1844664 --- keystone/common/policies/project_endpoint.py | 70 ++- .../protection/v3/test_project_endpoint.py | 465 ++++++++++++++++++ .../notes/bug-1844664-905cf6cad2e032a7.yaml | 36 ++ 3 files changed, 561 insertions(+), 10 deletions(-) create mode 100644 keystone/tests/protection/v3/test_project_endpoint.py create mode 100644 releasenotes/notes/bug-1844664-905cf6cad2e032a7.yaml diff --git a/keystone/common/policies/project_endpoint.py b/keystone/common/policies/project_endpoint.py index bc7bb77bba..c04cddd4d6 100644 --- a/keystone/common/policies/project_endpoint.py +++ b/keystone/common/policies/project_endpoint.py @@ -10,15 +10,50 @@ # License for the specific language governing permissions and limitations # under the License. +from oslo_log import versionutils from oslo_policy import policy from keystone.common.policies import base +deprecated_list_projects_for_endpoint = policy.DeprecatedRule( + name=base.IDENTITY % 'list_projects_for_endpoint', + check_str=base.RULE_ADMIN_REQUIRED, +) + +deprecated_add_endpoint_to_project = policy.DeprecatedRule( + name=base.IDENTITY % 'add_endpoint_to_project', + check_str=base.RULE_ADMIN_REQUIRED, +) + +deprecated_check_endpoint_in_project = policy.DeprecatedRule( + name=base.IDENTITY % 'check_endpoint_in_project', + check_str=base.RULE_ADMIN_REQUIRED, +) + +deprecated_list_endpoints_for_project = policy.DeprecatedRule( + name=base.IDENTITY % 'list_endpoints_for_project', + check_str=base.RULE_ADMIN_REQUIRED, +) + +deprecated_remove_endpoint_from_project = policy.DeprecatedRule( + name=base.IDENTITY % 'remove_endpoint_from_project', + check_str=base.RULE_ADMIN_REQUIRED, +) + +DEPRECATED_REASON = """ +As of the Train release, the project endpoint API now understands default +roles and system-scoped tokens, making the API more granular by default without +compromising security. The new policy defaults account for these changes +automatically. Be sure to take these new defaults into consideration if you are +relying on overrides in your deployment for the project endpoint API. +""" + + project_endpoint_policies = [ policy.DocumentedRuleDefault( name=base.IDENTITY % 'list_projects_for_endpoint', - check_str=base.RULE_ADMIN_REQUIRED, + check_str=base.SYSTEM_READER, # NOTE(lbragstad): While projects can be considered project-level APIs # with hierarchical multi-tenancy, endpoints are a system-level # resource. Managing associations between projects and endpoints should @@ -27,18 +62,24 @@ project_endpoint_policies = [ description='List projects allowed to access an endpoint.', operations=[{'path': ('/v3/OS-EP-FILTER/endpoints/{endpoint_id}/' 'projects'), - 'method': 'GET'}]), + 'method': 'GET'}], + deprecated_rule=deprecated_list_projects_for_endpoint, + deprecated_reason=DEPRECATED_REASON, + deprecated_since=versionutils.deprecated.TRAIN), policy.DocumentedRuleDefault( name=base.IDENTITY % 'add_endpoint_to_project', - check_str=base.RULE_ADMIN_REQUIRED, + check_str=base.SYSTEM_ADMIN, scope_types=['system'], description='Allow project to access an endpoint.', operations=[{'path': ('/v3/OS-EP-FILTER/projects/{project_id}/' 'endpoints/{endpoint_id}'), - 'method': 'PUT'}]), + 'method': 'PUT'}], + deprecated_rule=deprecated_add_endpoint_to_project, + deprecated_reason=DEPRECATED_REASON, + deprecated_since=versionutils.deprecated.TRAIN), policy.DocumentedRuleDefault( name=base.IDENTITY % 'check_endpoint_in_project', - check_str=base.RULE_ADMIN_REQUIRED, + check_str=base.SYSTEM_READER, scope_types=['system'], description='Check if a project is allowed to access an endpoint.', operations=[{'path': ('/v3/OS-EP-FILTER/projects/{project_id}/' @@ -46,24 +87,33 @@ project_endpoint_policies = [ 'method': 'GET'}, {'path': ('/v3/OS-EP-FILTER/projects/{project_id}/' 'endpoints/{endpoint_id}'), - 'method': 'HEAD'}]), + 'method': 'HEAD'}], + deprecated_rule=deprecated_check_endpoint_in_project, + deprecated_reason=DEPRECATED_REASON, + deprecated_since=versionutils.deprecated.TRAIN), policy.DocumentedRuleDefault( name=base.IDENTITY % 'list_endpoints_for_project', - check_str=base.RULE_ADMIN_REQUIRED, + check_str=base.SYSTEM_READER, scope_types=['system'], description='List the endpoints a project is allowed to access.', operations=[{'path': ('/v3/OS-EP-FILTER/projects/{project_id}/' 'endpoints'), - 'method': 'GET'}]), + 'method': 'GET'}], + deprecated_rule=deprecated_list_endpoints_for_project, + deprecated_reason=DEPRECATED_REASON, + deprecated_since=versionutils.deprecated.TRAIN), policy.DocumentedRuleDefault( name=base.IDENTITY % 'remove_endpoint_from_project', - check_str=base.RULE_ADMIN_REQUIRED, + check_str=base.SYSTEM_ADMIN, scope_types=['system'], description=('Remove access to an endpoint from a project that has ' 'previously been given explicit access.'), operations=[{'path': ('/v3/OS-EP-FILTER/projects/{project_id}/' 'endpoints/{endpoint_id}'), - 'method': 'DELETE'}]) + 'method': 'DELETE'}], + deprecated_rule=deprecated_remove_endpoint_from_project, + deprecated_reason=DEPRECATED_REASON, + deprecated_since=versionutils.deprecated.TRAIN), ] diff --git a/keystone/tests/protection/v3/test_project_endpoint.py b/keystone/tests/protection/v3/test_project_endpoint.py new file mode 100644 index 0000000000..4e2d5a14fd --- /dev/null +++ b/keystone/tests/protection/v3/test_project_endpoint.py @@ -0,0 +1,465 @@ +# Licensed under the Apache License, Version 2.0 (the "License"); you may +# not use this file except in compliance with the License. You may obtain +# a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT +# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the +# License for the specific language governing permissions and limitations +# under the License. + +import uuid + +from oslo_serialization import jsonutils +from six.moves import http_client + +from keystone.common.policies import base as bp +from keystone.common import provider_api +import keystone.conf +from keystone.tests.common import auth as common_auth +from keystone.tests import unit +from keystone.tests.unit import base_classes +from keystone.tests.unit import ksfixtures +from keystone.tests.unit.ksfixtures import temporaryfile + +CONF = keystone.conf.CONF +PROVIDERS = provider_api.ProviderAPIs + + +class _SystemUserProjectEndpointTests(object): + """Common default functionality for all system users.""" + + def test_user_can_list_projects_for_endpoint(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + + PROVIDERS.catalog_api.add_endpoint_to_project(endpoint['id'], project['id']) + with self.test_client() as c: + r = c.get('/v3/OS-EP-FILTER/endpoints/%s/projects' % endpoint['id'], + headers=self.headers) + for project_itr in r.json['projects']: + self.assertIn(project['id'], project_itr['id']) + + def test_user_can_check_endpoint_in_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + + PROVIDERS.catalog_api.add_endpoint_to_project(endpoint['id'], project['id']) + with self.test_client() as c: + c.get('/v3/OS-EP-FILTER/projects/%s/endpoints/%s' + % (project['id'], endpoint['id']), + headers=self.headers, + expected_status_code=http_client.NO_CONTENT) + + def test_user_can_list_endpoints_for_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + + PROVIDERS.catalog_api.add_endpoint_to_project(endpoint['id'], project['id']) + with self.test_client() as c: + r = c.get('/v3/OS-EP-FILTER/projects/%s/endpoints' % project['id'], + headers=self.headers) + for endpoint_itr in r.json['endpoints']: + self.assertIn(endpoint['id'], endpoint_itr['id']) + + +class _SystemReaderAndMemberProjectEndpointTests(object): + + def test_user_cannot_add_endpoint_to_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + with self.test_client() as c: + c.put('/v3/OS-EP-FILTER/projects/%s/endpoints/%s' + % (project['id'], endpoint['id']), + headers=self.headers, + expected_status_code=http_client.FORBIDDEN) + + def test_user_cannot_remove_endpoint_from_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + with self.test_client() as c: + c.delete('/v3/OS-EP-FILTER/projects/%s/endpoints/%s' + % (project['id'], endpoint['id']), + headers=self.headers, + expected_status_code=http_client.FORBIDDEN) + + +class _DomainAndProjectUserProjectEndpointTests(object): + + def test_user_cannot_list_projects_for_endpoint(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + + PROVIDERS.catalog_api.add_endpoint_to_project(endpoint['id'], project['id']) + with self.test_client() as c: + c.get('/v3/OS-EP-FILTER/endpoints/%s/projects' % endpoint['id'], + headers=self.headers, + expected_status_code=http_client.FORBIDDEN) + + def test_user_cannot_check_endpoint_in_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + + PROVIDERS.catalog_api.add_endpoint_to_project(endpoint['id'], project['id']) + with self.test_client() as c: + c.get('/v3/OS-EP-FILTER/projects/%s/endpoints/%s' + % (project['id'], endpoint['id']), + headers=self.headers, + expected_status_code=http_client.FORBIDDEN) + + def test_user_cannot_list_endpoints_for_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + + PROVIDERS.catalog_api.add_endpoint_to_project(endpoint['id'], project['id']) + with self.test_client() as c: + c.get('/v3/OS-EP-FILTER/projects/%s/endpoints' % project['id'], + headers=self.headers, + expected_status_code=http_client.FORBIDDEN) + + +class SystemReaderTests(base_classes.TestCaseWithBootstrap, + common_auth.AuthTestMixin, + _SystemUserProjectEndpointTests, + _SystemReaderAndMemberProjectEndpointTests): + + def setUp(self): + super(SystemReaderTests, self).setUp() + self.loadapp() + self.useFixture(ksfixtures.Policy(self.config_fixture)) + self.config_fixture.config(group='oslo_policy', enforce_scope=True) + + system_reader = unit.new_user_ref( + domain_id=CONF.identity.default_domain_id + ) + self.user_id = PROVIDERS.identity_api.create_user( + system_reader + )['id'] + PROVIDERS.assignment_api.create_system_grant_for_user( + self.user_id, self.bootstrapper.reader_role_id + ) + + auth = self.build_authentication_request( + user_id=self.user_id, password=system_reader['password'], + system=True + ) + + # Grab a token using the persona we're testing and prepare headers + # for requests we'll be making in the tests. + with self.test_client() as c: + r = c.post('/v3/auth/tokens', json=auth) + self.token_id = r.headers['X-Subject-Token'] + self.headers = {'X-Auth-Token': self.token_id} + + +class SystemMemberTests(base_classes.TestCaseWithBootstrap, + common_auth.AuthTestMixin, + _SystemUserProjectEndpointTests, + _SystemReaderAndMemberProjectEndpointTests): + + def setUp(self): + super(SystemMemberTests, self).setUp() + self.loadapp() + self.useFixture(ksfixtures.Policy(self.config_fixture)) + self.config_fixture.config(group='oslo_policy', enforce_scope=True) + + system_member = unit.new_user_ref( + domain_id=CONF.identity.default_domain_id + ) + self.user_id = PROVIDERS.identity_api.create_user( + system_member + )['id'] + PROVIDERS.assignment_api.create_system_grant_for_user( + self.user_id, self.bootstrapper.member_role_id + ) + + auth = self.build_authentication_request( + user_id=self.user_id, password=system_member['password'], + system=True + ) + + # Grab a token using the persona we're testing and prepare headers + # for requests we'll be making in the tests. + with self.test_client() as c: + r = c.post('/v3/auth/tokens', json=auth) + self.token_id = r.headers['X-Subject-Token'] + self.headers = {'X-Auth-Token': self.token_id} + + +class SystemAdminTests(base_classes.TestCaseWithBootstrap, + common_auth.AuthTestMixin, + _SystemUserProjectEndpointTests): + + def setUp(self): + super(SystemAdminTests, self).setUp() + self.loadapp() + self.useFixture(ksfixtures.Policy(self.config_fixture)) + self.config_fixture.config(group='oslo_policy', enforce_scope=True) + + # Reuse the system administrator account created during + # ``keystone-manage bootstrap`` + self.user_id = self.bootstrapper.admin_user_id + auth = self.build_authentication_request( + user_id=self.user_id, + password=self.bootstrapper.admin_password, + system=True + ) + + # Grab a token using the persona we're testing and prepare headers + # for requests we'll be making in the tests. + with self.test_client() as c: + r = c.post('/v3/auth/tokens', json=auth) + self.token_id = r.headers['X-Subject-Token'] + self.headers = {'X-Auth-Token': self.token_id} + + def test_user_can_add_endpoint_to_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + with self.test_client() as c: + c.put('/v3/OS-EP-FILTER/projects/%s/endpoints/%s' + % (project['id'], endpoint['id']), + headers=self.headers, + expected_status_code=http_client.NO_CONTENT) + + def test_user_can_remove_endpoint_from_project(self): + project = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, + unit.new_project_ref(domain_id=CONF.identity.default_domain_id) + ) + + service = PROVIDERS.catalog_api.create_service( + uuid.uuid4().hex, unit.new_service_ref() + ) + endpoint = unit.new_endpoint_ref(service['id'], region_id=None) + endpoint = PROVIDERS.catalog_api.create_endpoint( + endpoint['id'], endpoint + ) + PROVIDERS.catalog_api.add_endpoint_to_project(endpoint['id'], project['id']) + with self.test_client() as c: + c.delete('/v3/OS-EP-FILTER/projects/%s/endpoints/%s' + % (project['id'], endpoint['id']), + headers=self.headers, + expected_status_code=http_client.NO_CONTENT) + + +class DomainUserTests(base_classes.TestCaseWithBootstrap, + common_auth.AuthTestMixin, + _DomainAndProjectUserProjectEndpointTests, + _SystemReaderAndMemberProjectEndpointTests): + + def setUp(self): + super(DomainUserTests, self).setUp() + self.loadapp() + self.useFixture(ksfixtures.Policy(self.config_fixture)) + self.config_fixture.config(group='oslo_policy', enforce_scope=True) + + domain = PROVIDERS.resource_api.create_domain( + uuid.uuid4().hex, unit.new_domain_ref() + ) + self.domain_id = domain['id'] + domain_admin = unit.new_user_ref(domain_id=self.domain_id) + self.user_id = PROVIDERS.identity_api.create_user(domain_admin)['id'] + PROVIDERS.assignment_api.create_grant( + self.bootstrapper.admin_role_id, user_id=self.user_id, + domain_id=self.domain_id + ) + + auth = self.build_authentication_request( + user_id=self.user_id, + password=domain_admin['password'], + domain_id=self.domain_id + ) + + # Grab a token using the persona we're testing and prepare headers + # for requests we'll be making in the tests. + with self.test_client() as c: + r = c.post('/v3/auth/tokens', json=auth) + self.token_id = r.headers['X-Subject-Token'] + self.headers = {'X-Auth-Token': self.token_id} + + +class ProjectUserTests(base_classes.TestCaseWithBootstrap, + common_auth.AuthTestMixin, + _DomainAndProjectUserProjectEndpointTests, + _SystemReaderAndMemberProjectEndpointTests): + + def setUp(self): + super(ProjectUserTests, self).setUp() + self.loadapp() + self.useFixture(ksfixtures.Policy(self.config_fixture)) + self.config_fixture.config(group='oslo_policy', enforce_scope=True) + + self.user_id = self.bootstrapper.admin_user_id + auth = self.build_authentication_request( + user_id=self.user_id, + password=self.bootstrapper.admin_password, + project_id=self.bootstrapper.project_id + ) + + # Grab a token using the persona we're testing and prepare headers + # for requests we'll be making in the tests. + with self.test_client() as c: + r = c.post('/v3/auth/tokens', json=auth) + self.token_id = r.headers['X-Subject-Token'] + self.headers = {'X-Auth-Token': self.token_id} + + +class ProjectUserTestsWithoutEnforceScope( + base_classes.TestCaseWithBootstrap, + common_auth.AuthTestMixin, + _DomainAndProjectUserProjectEndpointTests, + _SystemReaderAndMemberProjectEndpointTests): + + def _override_policy(self): + # TODO(cmurphy): Remove this once the deprecated policies in + # keystone.common.policies.project_endpoint have been removed. This is + # only here to make sure we test the new policies instead of the + # deprecated ones. Oslo.policy will OR deprecated policies with new + # policies to maintain compatibility and give operators a chance to + # update permissions or update policies without breaking users. This + # will cause these specific tests to fail since we're trying to correct + # this broken behavior with better scope checking. + with open(self.policy_file_name, 'w') as f: + overridden_policies = { + 'identity:list_projects_for_endpoint': bp.SYSTEM_READER, + 'identity:add_endpoint_to_project': bp.SYSTEM_ADMIN, + 'identity:check_endpoint_in_project': bp.SYSTEM_READER, + 'identity:list_endpoints_for_project': bp.SYSTEM_READER, + 'identity:remove_endpoint_from_project': bp.SYSTEM_ADMIN + } + f.write(jsonutils.dumps(overridden_policies)) + + def setUp(self): + super(ProjectUserTestsWithoutEnforceScope, self).setUp() + self.loadapp() + self.policy_file = self.useFixture(temporaryfile.SecureTempFile()) + self.policy_file_name = self.policy_file.file_name + self.useFixture( + ksfixtures.Policy( + self.config_fixture, policy_file=self.policy_file_name + ) + ) + self._override_policy() + # Explicity set enforce_scope to False to make sure we maintain + # backwards compatibility with project users. + self.config_fixture.config(group='oslo_policy', enforce_scope=False) + + domain = PROVIDERS.resource_api.create_domain( + uuid.uuid4().hex, unit.new_domain_ref() + ) + user = unit.new_user_ref(domain_id=domain['id']) + self.user_id = PROVIDERS.identity_api.create_user(user)['id'] + + self.project_id = PROVIDERS.resource_api.create_project( + uuid.uuid4().hex, unit.new_project_ref(domain_id=domain['id']) + )['id'] + + PROVIDERS.assignment_api.create_grant( + self.bootstrapper.admin_role_id, user_id=self.user_id, + project_id=self.project_id + ) + + auth = self.build_authentication_request( + user_id=self.user_id, + password=user['password'], + project_id=self.project_id + ) + + # Grab a token using the persona we're testing and prepare headers + # for requests we'll be making in the tests. + with self.test_client() as c: + r = c.post('/v3/auth/tokens', json=auth) + self.token_id = r.headers['X-Subject-Token'] + self.headers = {'X-Auth-Token': self.token_id} diff --git a/releasenotes/notes/bug-1844664-905cf6cad2e032a7.yaml b/releasenotes/notes/bug-1844664-905cf6cad2e032a7.yaml new file mode 100644 index 0000000000..ed28fba432 --- /dev/null +++ b/releasenotes/notes/bug-1844664-905cf6cad2e032a7.yaml @@ -0,0 +1,36 @@ +--- +features: + - | + [`bug 1844664 `_] + The Project Endpoints API now supports the ``admin``, + ``member``, and ``reader`` default roles. + +upgrade: + - | + [`bug 1844664 `_] + The Project Endpoints API uses new default policies to + make it more accessible to end users and administrators in a secure way. + Please consider these new defaults if your deployment overrides Project + Endpoints policies. +deprecations: + - | + [`bug 1844664 `_] + The Project Endpoints policies have been deprecated. The + ``identity:list_projects_for_endpoint`` now use ``(role:reader and system_scope:all)`` + ``identity:check_endpoint_in_project`` policies now use + ``role:reader and system_scope:all`` and ``identity:list_endpoints_for_project`` + now use ``(role:reader and system_scope:all)`` instead of + ``rule:admin_required``. The ``identity:add_endpoint_to_project`` now use + ``(role:admin and system_scope:all)`` + instead of ``rule:admin_required``and ``identity:remove_endpoint_from_project`` + policies now use ``role:admin and system_scope:all`` instead of + ``rule:admin_required``. + These new defaults automatically account for system-scope and support + a read-only role, making it easier for system administrators to delegate + subsets of responsibility without compromising security. Please consider + these new defaults if your deployment overrides the Project Endpoints policies. +security: + - | + [`bug 1844664 `_] + The Project Endpoints API now uses system-scope and default + roles to provide better accessibility to users in a secure manner.