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

Wrong endpoint on the TLS secret example #5817

Merged
merged 2 commits into from Nov 12, 2019
Merged

Conversation

yacinelazaar
Copy link
Contributor

@yacinelazaar yacinelazaar commented Nov 12, 2019

This snippet provided in that section uses 'web' as an endpoint instead of 'websecure'.

What does this PR do?

Motivation

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

@dduportal
Copy link
Contributor

Hi @yacinelazaar , if my understanding of https://traefik.slack.com/archives/C0CDT22PJ/p1573560589045800 is correct: the snippet you changed is referenced in https://docs.traefik.io/v2.0/routing/providers/kubernetes-crd/#middleware, and does not enable TLS: so the entry point has to be web (it is the port 80).

If you meant https://docs.traefik.io/routing/providers/kubernetes-crd/#tls, can you add websecure to the entry points list (and not replace the existing one) please?

Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see my message below

This snippet provided in that section uses 'web' as an endpoint instead of 'websecure'.
Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed with @yacinelazaar , the entrypoint is finally replaced (instead of added ) to stay coherent with https://docs.traefik.io/v2.0/user-guides/docker-compose/acme-tls/ for example.

LGTM 👍

@ldez ldez added bot/light-review decreases the number of required LGTM from 3 to 1. and removed status/2-needs-review labels Nov 12, 2019
@ldez ldez added this to the 2.0 milestone Nov 12, 2019
@ldez ldez added this to To review in v2 via automation Nov 12, 2019
@traefiker traefiker merged commit 587d3f9 into traefik:v2.0 Nov 12, 2019
v2 automation moved this from To review to Done Nov 12, 2019
@traefiker traefiker removed bot/light-review decreases the number of required LGTM from 3 to 1. status/3-needs-merge labels Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v2
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants