DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Change oauth redirect to 307

Browse source
This commit is contained in:
Bastien Wermeille 2025-01-07 15:22:17 +01:00
parent 69fd353086
commit efd71e75a8
3 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
rootfs/etc/nginx/template/nginx.tmpl
View file

@ -1104,7 +1104,7 @@ stream {
modsecurity off;
{{ end }}
return 302 {{ buildAuthSignURL $externalAuth.SigninURL $externalAuth.SigninURLRedirectParam }};
return 307 {{ buildAuthSignURL $externalAuth.SigninURL $externalAuth.SigninURLRedirectParam }};
}
{{ end }}
{{ end }}

6
test/e2e/annotations/auth.go
View file

@ -494,7 +494,7 @@ http {
WithQuery("a", "b").
WithQuery("c", "d").
Expect().
Status(http.StatusFound).
Status(http.StatusTemporaryRedirect).
Header("Location").Equal(fmt.Sprintf("http://%s/auth/start?rd=http://%s%s", host, host, url.QueryEscape("/?a=b&c=d")))
})
@ -714,7 +714,7 @@ http {
WithQuery("a", "b").
WithQuery("c", "d").
Expect().
Status(http.StatusFound).
Status(http.StatusTemporaryRedirect).
Header("Location").Equal(fmt.Sprintf("http://%s/auth/start?orig=http://%s%s", host, host, url.QueryEscape("/?a=b&c=d")))
})
@ -861,7 +861,7 @@ http {
WithQuery("a", "b").
WithQuery("c", "d").
Expect().
Status(http.StatusFound).
Status(http.StatusTemporaryRedirect).
Header("Location").Equal(fmt.Sprintf("http://%s/auth/start?rd=http://%s%s", thisHost, thisHost, url.QueryEscape("/?a=b&c=d")))
})
})

2
test/e2e/annotations/satisfy.go
View file

@ -121,7 +121,7 @@ var _ = framework.DescribeAnnotation("satisfy", func() {
WithQuery("a", "b").
WithQuery("c", "d").
Expect().
Status(http.StatusFound).
Status(http.StatusTemporaryRedirect).
Header("Location").Equal(fmt.Sprintf("http://%s/auth/start?rd=http://%s%s", host, host, url.QueryEscape("/?a=b&c=d")))
})
})