Use UpdateService in pipeline trigger token endpoints
What does this MR do and why?
Part of #421383 (closed).
This MR is based off of !130449 (comment 1635909092) and addresses bullet
Since we are making many changes and I'm getting a bit nervous with consolidating permissions everywhere, what do you think of doing the followings:
- In this MR we continue refining the 3 service objects.
- In this MR we use
CreateService
in the 3 endpoints (Controller, REST, GraphQL) - and adjust permissions to be consistent.- In a separate MR we use
UpdateService
in the 3 endpoints (Controller, REST, GraphQL) - and adjust permissions to be consistent.- In a separate MR we use
DestroyService
in the 3 endpoints (Controller, REST, GraphQL) - and adjust permissions to be consistent.
Screenshots or screen recordings
Please see the description of !130449 (closed) for more information.
How to set up and validate locally
Please see the description of !130449 (closed) for more information.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Missy Davies