gitlab-ce-ldap-sync icon indicating copy to clipboard operation
gitlab-ce-ldap-sync copied to clipboard

Passwords should not be set

Open frafra opened this issue 6 years ago • 2 comments

Passwords should not be set for LDAP users. GitLab API shows that password is an optional field: https://docs.gitlab.com/ce/api/users.html#user-creation

frafra avatar Jun 24 '19 13:06 frafra

Hi Fran,

I agree, though I had to have a password set for new users or the Gitlab API would reject the creation.

From 818c52cd68c59b213fd44e737d7e9a02d0375e56

generateRandomPassword(): New helper function to generate fairly random passwords for new users, as a user for some reason can't be created without a password. (They'll never need it though!) -- See https://gitlab.com/gitlab-org/gitlab-ce/issues/55426

Adambean avatar Jun 24 '19 16:06 Adambean

Ok, thanks for reporting it.

frafra avatar Jun 25 '19 07:06 frafra