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

only add --certificate-authority if either k8s_ca_uri or k8s_ca_pem i… #88

Merged
merged 1 commit into from
Mar 10, 2019

Conversation

nrobert13
Copy link
Contributor

…s provided.

@nabadger nabadger merged commit 641bcde into mintel:master Mar 10, 2019
@nrobert13 nrobert13 deleted the fix_template_public_ca branch March 13, 2019 10:12
@nrobert13 nrobert13 restored the fix_template_public_ca branch March 13, 2019 10:12
@nickmintel
Copy link
Contributor

nickmintel commented Mar 18, 2019

@bjornryden out of interest do you use dex-auth via windows? I'm looking at testing #86 ...but as far as I'm aware none of the existing commands (in master) on the windows tab work.

@bjornryden
Copy link

Hi @nickmintel. No, it was just because I spotted that the --certificate-authority was still present on that tab.

@nrobert13
Copy link
Contributor Author

@nickmintel, I have colleagues successfully using the command printed out in the windows tab in cmd.

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.

4 participants