yahoo-eng-team team mailing list archive

[Ghanshyam Mann <gmann@xxxxxxxxxxxxxxxxx>]

Public bug reported:

os-availability-zone API policy is default to admin_or_owner[1] but API
is allowed for everyone.

This is because API does not pass the project_id in policy target so that oslo policy can decide the ownership. 
https://github.com/openstack/nova/blob/1fcd74730d343b7cee12a0a50ea537dc4ff87f65/nova/api/openstack/compute/availability_zone.py#L111

and if no target is passed then, policy.py add the default targets which is nothing but context.project_id (allow for everyone try to access)
- https://github.com/openstack/nova/blob/c16315165ce307c605cf4b608b2df3aa06f46982/nova/policy.py#L191

We need to default the GET /os-availability-zone policy same as GET
details /os-availability-zone which is admin.

[1]
- https://github.com/openstack/nova/blob/1fcd74730d343b7cee12a0a50ea537dc4ff87f65/nova/policies/availability_zone.py#L27

** Affects: nova
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1862484

Title:
  GET os-availability-zone API policy is allowed for everyone even
  policy defaults is admin_or_owner

Status in OpenStack Compute (nova):
  New

Bug description:
  os-availability-zone API policy is default to admin_or_owner[1] but
  API is allowed for everyone.

  This is because API does not pass the project_id in policy target so that oslo policy can decide the ownership. 
  https://github.com/openstack/nova/blob/1fcd74730d343b7cee12a0a50ea537dc4ff87f65/nova/api/openstack/compute/availability_zone.py#L111

  and if no target is passed then, policy.py add the default targets which is nothing but context.project_id (allow for everyone try to access)
  - https://github.com/openstack/nova/blob/c16315165ce307c605cf4b608b2df3aa06f46982/nova/policy.py#L191

  We need to default the GET /os-availability-zone policy same as GET
  details /os-availability-zone which is admin.

  [1]
  - https://github.com/openstack/nova/blob/1fcd74730d343b7cee12a0a50ea537dc4ff87f65/nova/policies/availability_zone.py#L27

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1862484/+subscriptions