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

feat: add support for custom TLS certificate #212

Merged
merged 1 commit into from
Dec 21, 2018
Merged

Conversation

hbagdi
Copy link
Member

@hbagdi hbagdi commented Dec 18, 2018

Following options have been added to work with custom TLS certificates,
SNI names and to ignore a self-signed certificate:

  • --admin-tls-skip-verify: to skip validation of a certificate, this
    shouldn't be used in a production environment.
  • --admin-tls-server-name: use this if the FQDN of Kong's Admin API
    doesn't match the SNI name in the certificate.
  • --admin-ca-cert-file: use this to specify a custom CA cert which is
    not part of the bundled CA certs.

Following options have been added to work with custom TLS certificates,
SNI names and to ignore a self-signed certificate:

- `--admin-tls-skip-verify`: to skip validation of a certificate, this
shouldn't be used in a production environment.
- `--admin-tls-server-name`: use this if the FQDN of Kong's Admin API
doesn't match the SNI name in the certificate.
- `--admin-ca-cert-file`: use this to specify a custom CA cert which is
not part of the bundled CA certs.
@codecov-io
Copy link

Codecov Report

Merging #212 into master will increase coverage by 0.15%.
The diff coverage is 34.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #212      +/-   ##
==========================================
+ Coverage    21.7%   21.85%   +0.15%     
==========================================
  Files          24       24              
  Lines        2350     2375      +25     
==========================================
+ Hits          510      519       +9     
- Misses       1796     1812      +16     
  Partials       44       44
Impacted Files Coverage Δ
internal/ingress/controller/controller.go 0% <ø> (ø) ⬆️
cli/ingress-controller/main.go 0% <0%> (ø) ⬆️
cli/ingress-controller/flags.go 81.9% <100%> (+1.69%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78699ba...532150a. Read the comment docs.

@hbagdi hbagdi merged commit 12d1e52 into master Dec 21, 2018
@hbagdi hbagdi deleted the feat/tls-configs branch December 21, 2018 00:34
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.

None yet

3 participants