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

WIP: Move certificate authentication from location to server #939

Closed
wants to merge 1 commit into from

Conversation

aledbf
Copy link
Member

@aledbf aledbf commented Jul 6, 2017

Replaces #603

@gianrubio I tried to rebase from your PR without success.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 6, 2017
@k8s-reviewable
Copy link

This change is Reviewable

@aledbf aledbf changed the title Move certificate authentication from location to server WIP: Move certificate authentication from location to server Jul 6, 2017
@gianrubio
Copy link
Contributor

LGTM

@filipows
Copy link

filipows commented Aug 8, 2017

Hi,

any news about that?

@aledbf
Copy link
Member Author

aledbf commented Aug 23, 2017

@filipows can you test the image quay.io/aledbf/nginx-ingress-controller:0.189 please?

@aledbf aledbf deleted the cauth-cert branch August 27, 2017 19:17
@filipows
Copy link

It's working great. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants