Skip to content

Bump min access level for Bulk Import UI to Owner when listing groups

George Koltsov requested to merge georgekoltsov/bulk-import-min-access into master

What does this MR do?

This MR bumps min access level to owner when listing groups available to import when using bulk import feature https://docs.gitlab.com/ee/user/group/import/

This is a bug fix, as the actual import background process requires owner privileges. The bulk import feature is disabled by default for self managed installations, as it is a beta/experimental feature that is still under construction. Doing this change does not have compatibility implications.

Mentions #331704 (closed)

Screenshots (strongly suggested)

owner

In order to test locally, make sure to enable bulk_import feature flag.

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
Edited by George Koltsov

Merge request reports

Loading