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

mTLS Kong Admin API authentication support #1958

Closed
1 task done
ludovic-pourrat opened this issue Oct 21, 2021 · 4 comments · Fixed by #2148 or #2314
Closed
1 task done

mTLS Kong Admin API authentication support #1958

ludovic-pourrat opened this issue Oct 21, 2021 · 4 comments · Fixed by #2148 or #2314
Assignees

Comments

@ludovic-pourrat
Copy link
Contributor

ludovic-pourrat commented Oct 21, 2021

Is there an existing issue for this?

  • I have searched the existing issues

Problem Statement

This is a new feature request to support Kong mTLS authentication capabilities.

Kong API administration API can be secured via mTLS authentication by setting the following NGINX directives,

    nginx_admin_proxy_ssl_verify: "on"
    nginx_admin_ssl_client_certificate: /etc/secrets/tls.crt
    nginx_admin_ssl_verify_client: "on"
    nginx_admin_ssl_verify_depth: 3
    nginx_admin_ssl_trusted_certificate: /etc/secrets/ca/ca-bundle.crt

Thus, the ingress controller should provide a way to specify client certificates.

Proposed Solution

Add new arguments to specify the client certificate and key and setup the http/client package to use client certificates.

Additional information

No response

Acceptance Criteria

Once mTLS activated the ingress controller should authenticate to Kong administrative API with the specified client certificate and key.

ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Oct 21, 2021
Feature request proposal related to mTLS Kong Admin API authentication support Kong#1958
@shaneutt
Copy link
Contributor

shaneutt commented Dec 6, 2021

@ludovic-pourrat the PR for this timed out due to inactivity, would you want that re-opened? Is there any additional assistance you need from us to continue it?

@ludovic-pourrat
Copy link
Contributor Author

I will work on it next week. sorry for the delay.

@shaneutt
Copy link
Contributor

shaneutt commented Dec 6, 2021

No worries, thanks for the update and please let us know how we can help. Whenever you're ready to get back to it, feel free to just "re-open" the PR 👍

@stale
Copy link

stale bot commented Dec 20, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Will be closed unless advocated for within 7 days label Dec 20, 2021
@stale stale bot closed this as completed Dec 27, 2021
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 9, 2022
Feature request proposal related to mTLS Kong Admin API authentication support Kong#1958
Updated according feedback.
@mflendrich mflendrich reopened this Jan 19, 2022
@stale stale bot removed the stale Will be closed unless advocated for within 7 days label Jan 19, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 26, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 26, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 27, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 27, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 27, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 28, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 28, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Jan 28, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Feb 3, 2022
ludovic-pourrat added a commit to ludovic-pourrat/kubernetes-ingress-controller that referenced this issue Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment