Skip to content

Allow edits to application settings in maintenance mode

Aakriti Gupta requested to merge ag-allow-admin-settings-change into master

What does this MR do?

Closes #296536 (closed)

This MR makes it so that admins can edit all application settings and not just the ones edited through api v4.

Currently, only maintenance mode is edited through api v4.

It also adds in a test to assert that application settings cannot be edited on Geo secondaries, since we want this only on a primary.

Screenshots (strongly suggested)

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 Aakriti Gupta

Merge request reports