RBAC rules are applied incorrectly against AZ objects

Bug #1955023 reported by Vladimir Grevtsev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Triaged
Medium
Unassigned
3.3
Triaged
Medium
Unassigned

Bug Description

=== Environment

MAAS+Candid+RBAC

candid v1.8.1 799 latest/stable canonical* -
canonical-rbac 1.0.2-400-g.6e658ad 224 - canonical* -
maas 3.1.0-10901-g.f1f8f1505 17913 3.1/stable canonical* -
maas-cli 0.6.5 13 latest/stable canonical* -

Candid's config file:

identity-providers:
- type: static
  name: static
  users:
    administrator:
      name: Administrator
      email: <email address hidden>
      password: admin
      groups:
       - group1
       - group2
    user:
      name: user
      email: <email address hidden>
      password: user
      groups:
       - group2

=== Problem description

User, authenticated with "user" credentials still could add a new AZ, despite the lack of permission (either "Auditor" or nothing assigned in "All DNS, AZ, Settings, Images" scope). However, this user can only add a new entry, but not modify or delete an existing one.

Revision history for this message
Vladimir Grevtsev (vlgrevtsev) wrote :

RBAC assignments attached.

Revision history for this message
Alberto Donato (ack) wrote :

Could you please provide which policies are applied to the user (or groups the user is part of) in the context of maas?

Changed in maas:
status: New → Incomplete
Revision history for this message
Vladimir Grevtsev (vlgrevtsev) wrote :

Hi Alberto,

Please, see screenshot attached in comment #1.
If that's not sufficient - please, let me know and I'll update the bug accordingly.

Changed in maas:
status: Incomplete → New
Revision history for this message
Alberto Donato (ack) wrote :

The zone websocket handler doesn't define create permissions (nor it uses a form), so no permission checks are performed.

Changed in maas:
status: New → Triaged
milestone: none → next
importance: Undecided → High
Changed in maas:
importance: High → Medium
milestone: next → 3.3.0
Changed in maas:
milestone: 3.3.0 → 3.4.0
Alberto Donato (ack)
Changed in maas:
milestone: 3.4.0 → 3.4.x
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.