Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add LDAP configuration properties for ATC #2480

Merged
merged 3 commits into from Aug 17, 2018
Merged

Conversation

JamesClonk
Copy link
Contributor

No description provided.

signed-off-by: Fabio Berchtold <fabio.berchtold@swisscom.com>
signed-off-by: Fabio Berchtold <fabio.berchtold@swisscom.com>
@JamesClonk
Copy link
Contributor Author

JamesClonk commented Aug 9, 2018

I've tested it with a self-built concourse dev-release, the LDAP feature of atc/dex seems to work out of the box. 👍

Copy link
Member

@vito vito left a comment

Choose a reason for hiding this comment

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

Thanks, looks like we missed all this! Noticed one problem so I left a comment.

@@ -0,0 +1 @@
<%= p("ldap_auth.ca_cert", "") %>

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

Signed-off-by: Fabio Berchtold <fabio.berchtold@swisscom.com>
@JamesClonk
Copy link
Contributor Author

@vito my bad, just saw the other merged PR (#2479).
Changed it accordingly. 😃

@vito
Copy link
Member

vito commented Aug 17, 2018

thanks!

@vito vito merged commit 1db50ee into concourse:master Aug 17, 2018
@vito vito modified the milestone: v4.1.0 Aug 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants