Skip to content

Commit

Permalink
modify the update permission for robot
Browse files Browse the repository at this point in the history
From the security perspective, to aviod the privilege escalation, the udpate permission will not in the robot scope.

1, remove the robot update permission from the scope.
2, anyone who has the robot update persmission can update the permission set of a robot.

Signed-off-by: wang yan <[email protected]>
  • Loading branch information
wy65701436 committed Oct 12, 2024
1 parent 471a36e commit a333781
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions proposals/new/Robot-Account-Expand.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@ Additionally, by recording the creator of each robot account in the database and
## Non Goal

1. No support for configuring the banned permissions in the harbor v2.10.0
2. No support for granting system configuration permission for a robot
2. No support for granting system configuration permission for a robot account
3. No support for updating a robot account using another robot account

## Terms
Creator robot account: a robot account that has the permission to create other robot accounts. It can initiate the creation of new robot accounts.
Expand Down Expand Up @@ -62,12 +63,7 @@ Creation: If a robot account has the permission to create robot accounts, it can
1. any project level robot account can be created by a system or project level robot account who with the robot creation permission.
2. any system level robot account can be created by a system level robot account who with the robot creation permission.

Update: A nested robot account cannot be assigned the permissions that exceed those of its creator.
1. any nested robot account can be updated by someone who has the robot update permission.
1. A human user with the relevant robot update permission.
2. The creator robot account of the nested robot.
3. The nested robot account itself (if granted the necessary permissions).
2. if the **creator robot account is removed**, the nested robot account will be **escalated** to an individual robot account. At that point, **anyone** with the appropriate robot update permission can manage it without any limitation.
Update: **Anyone** with the appropriate robot update permission can manage the robot account without any limitation.

**Note**: Since the creator robot account’s permissions can be updated without impacting its nested accounts, this can lead to situations where the **nested robot account has more powerful permissions** than its creator.

Expand Down Expand Up @@ -116,7 +112,6 @@ Examples:
| User-Group | Update | Y |
| User-Group | Delete | Y |
| Robot | Read | Y |
| Robot | Update | Y |
| Robot | List | Y |
| Robot | Create | Y |
| Robot | Delete | Y |
Expand Down

0 comments on commit a333781

Please sign in to comment.