Skip to content

Commit

Permalink
Replace builtin SSL redirection with manual one (#415)
Browse files Browse the repository at this point in the history
As outlined in this (traefik/traefik#1957)
issue SSL redirection with PathPrefixStrip does not work really well.

A solution was provided in this (traefik/traefik#3631) PR,
released in 1.7, but it didn't really solve the issue.

In fact, there were several subsequent issues opened
(traefik/traefik#3999, traefik/traefik#3876)
but they got closed.

Another issue was opened in the Traefik repo: traefik/traefik#4085

Until then this workaround provides the same functionality.
  • Loading branch information
sagikazarmark authored and ahma committed Oct 21, 2018
1 parent 5a854b2 commit 397c81a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion pipeline-cluster-monitor/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
description: A Helm chart for Kubernetes
name: pipeline-cluster-monitor
version: 0.1.8
version: 0.1.9
keywords:
- pipeline
- cluster
Expand Down
4 changes: 3 additions & 1 deletion pipeline-cluster-monitor/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,9 @@ grafana:
annotations:
kubernetes.io/ingress.class: traefik
traefik.frontend.rule.type: PathPrefixStrip
traefik.ingress.kubernetes.io/ssl-redirect: "true"
traefik.ingress.kubernetes.io/redirect-permanent: "true"
traefik.ingress.kubernetes.io/redirect-regex: ^http://(.*)
traefik.ingress.kubernetes.io/redirect-replacement: https://$1
path: /grafana
hosts:
- localhost
Expand Down

0 comments on commit 397c81a

Please sign in to comment.