Skip to content

Fix LDAP username issue#1746

Closed
frol2103 wants to merge 1 commit into
sameersbn:masterfrom
frol2103:fixLDAPUsername
Closed

Fix LDAP username issue#1746
frol2103 wants to merge 1 commit into
sameersbn:masterfrom
frol2103:fixLDAPUsername

Conversation

@frol2103

Copy link
Copy Markdown

Fixes #1745

I am not sure of the procedure to add an array in a env variable but I didn't find any example.

Please do not hesitate to tell if change needs to be made.

@solidnerd

Copy link
Copy Markdown
Collaborator

Hey @frol2103,

can you rebase against master to see the new ci system running on your improvement.

@solidnerd solidnerd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should rebase against master so we see CI running.

gjrtimmer referenced this pull request in gjrtimmer/docker-gitlab Dec 14, 2018
Fixes #699
Fixes #1746
Fixes #1814
@solidnerd solidnerd closed this in 33ef7ff Dec 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ldap issue with userid config

2 participants