This is a follow up to !28057 (merged). There has been some negative feedback around this change (see thread here: #209020 (closed)) and it probably should have been behind a feature flag from the get go (learning experience
group_administration_nav_item
feature flaggroup_saml
, enforced_sso
, and enforced_sso_requires_session
group_saml
in config/gitlab.yml
. See https://gitlab.com/gitlab-org/gitlab-development-kit/-/blob/master/doc/howto/saml.md#gitlab-configuration. Note: Ensure this is added to the development:
section of your configgroup_administration_nav_item
feature flag
bin/rails console
group = Group.find_by(name: 'group name')
Feature.enable(:group_administration_nav_item, group)
Page | Before !28057 (merged) was merged | After w/ feature disabled | After w/ feature enabled |
---|---|---|---|
Parent Group | ![]() |
![]() |
![]() |
Subgroup | ![]() |
![]() |
![]() |
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:
@gitlab-com/gl-security/appsec