Skip to content

Update permissions for Licenses and Licenses Controllers

Tetiana Chupryna requested to merge 13247-licenses-permissions into master

What does this MR do?

A part of #13247 (closed)

  1. We remove licenses_list license feature to simplify permissions. License compliance page feature is based on License Management aka License Compliance analizer so we just check if this second feature is available. Also, we add a condition for licenses_list feature flag, since this feature isn't publicly available yet. That will help to simplify rules and make fewer mistakes with permissions.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Tetiana Chupryna

Merge request reports