Skip to content

Disconnect package permission from repository

Steve Abrams requested to merge 332028-remove-repository-package-check into master

🔎 What does this MR do?

Currently, package permissions rely on repository permissions. This means if the repository is disabled for a project, the package registry also becomes inaccessible.

Packages do not depend on the repository, so this causes some confusion.

In this MR we update the package permission to not rely on the repository status.

This also aligns with the container registry permissions.

It seemed like overkill to add test examples to every package type, so I've chosen one to update here in addition to updating the policy spec.

Screenshots or Screencasts (strongly suggested)

How to setup and validate locally (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • [-] 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

Related to #332028 (closed)

Edited by Steve Abrams

Merge request reports