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.

Sign up for GitHub

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

Jump to bottom

Support ingress.kubernetes.io/force-ssl-redirect #63

Closed
davecheney opened this issue Nov 28, 2017 · 1 comment
Closed

Support ingress.kubernetes.io/force-ssl-redirect #63

davecheney opened this issue Nov 28, 2017 · 1 comment
Milestone
0.3.0

Comments

@davecheney
Copy link
Contributor

Add support for ingress.kubernetes.io/force-ssl-redirect per Ingress which will require creating a redirect route in the http version listener. This will need LDS to become a proper cache.
This was referenced Dec 15, 2017
Closed
Closed
@davecheney davecheney added this to the 0.3.0 milestone Dec 15, 2017
This was referenced Dec 15, 2017
Merged
Closed
@poonai poonai mentioned this issue Jan 13, 2018
Closed
@poonai
Copy link

poonai commented Jan 14, 2018 •
edited

while adding ingress, contour will check the listener cache whether the ingress domain has SSL or not. if it has SSL, contour will recompute vhost with SSL.

while adding secret, contour will check the ingress which is already present in the vhost or not. if it present, it'll recompute the vhost.

I feel that there should be some syncing mechanism for vhost in the translator.(I'm not sure)

@davecheney davecheney mentioned this issue Jan 15, 2018
Merged
davecheney added a commit to davecheney/contour that referenced this issue Jan 15, 2018
@davecheney
internal/contour: support ingress.kubernetes.io/force-ssl-redirect: true
5397e3e 
Fixes projectcontour#63
Closes projectcontour#146

Add support for the ingress.kubernetes.io/force-ssl-redirect: "true"
annotation

Signed-off-by: Dave Cheney <dave@cheney.net>
@davecheney davecheney mentioned this issue Jan 15, 2018
Merged
lrouquette added a commit to lrouquette/contour that referenced this issue Jul 13, 2021
@lrouquette
Merge pull request projectcontour#63 from lrouquette/v1.5-adobe-spdy
9738da8 
support SPDY protocol upgrade
sunjayBhatia pushed a commit that referenced this issue Jan 30, 2023
@danehans
Updates ConfigMap Name (#63)
c2305b3 
Signed-off-by: Daneyon Hansen <daneyonhansen@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.3.0
Development

Successfully merging a pull request may close this issue.

support added for ingress.kubernetes.io/force-ssl-redirect poonai/contour
2 participants
@davecheney @poonai