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

Expose TLS setup in /api/tls #4224

Closed
wants to merge 3 commits into from
Closed

Conversation

athoune
Copy link

@athoune athoune commented Nov 23, 2018

What does this PR do?

This PR add an entrypoint in the api for exposing tels configs.

Motivation

It's hard to understand troubles with https. I want to see what is done, not what I asked.

More

  • Added/updated tests
  • Added/updated documentation

@mmatur
Copy link
Member

mmatur commented Nov 23, 2018

Hi @athoune,

Thanks for your interest in the project.

Why do you want to expose your certificates in the API?

Few weeks ago we have created a PR to remove certificates content in the API #3665.

@athoune
Copy link
Author

athoune commented Nov 23, 2018

I'm fighting with tls configuration. I need some explicit certificates, some star certificates, and an ACME fallback. [file] works well for adding certificates.

But now, I'm fighting with some client side crash.
curl: (35) error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error

I'm seeking some visibility.

@athoune
Copy link
Author

athoune commented Nov 24, 2018

This PR goes in the wrong direction. I'll be back with a demo : https://github.com/factorysh/demo_traefik_cfssl.

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

4 participants