Skip to content
This repository has been archived by the owner on Sep 6, 2023. It is now read-only.

Use new ldap configuration method #220

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ndf-dw
Copy link

@ndf-dw ndf-dw commented Jul 25, 2023

Use new configuration method for LDAP.

The ldap_servers format is introduced in gitlab 14.7 and mandatory since gitlab 16.2 (released 3 days ago). Gitlab 16.2 won't work with old configuration.

Format is described in https://docs.gitlab.com/ee/administration/auth/ldap/

There are a few configuration settings not handle by this template, I just manage to set existing LDAP variables.

@ndf-dw ndf-dw force-pushed the UseNewLDAPConfigurationMethod branch from 1f78d33 to 105d247 Compare July 25, 2023 14:46
@CedricLevasseur
Copy link

Oh thanks you so much @ndf-dw, i was writing the same but struggling with molecule.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants