Change oauth redirect to 307

kubernetes · Jan 9, 2025 · efd71e7 · efd71e7
1 parent 69fd353
commit efd71e7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/rootfs/etc/nginx/template/nginx.tmpl b/rootfs/etc/nginx/template/nginx.tmpl
@@ -1104,7 +1104,7 @@ stream {
             modsecurity off;
             {{ end }}
 
-            return 302 {{ buildAuthSignURL $externalAuth.SigninURL $externalAuth.SigninURLRedirectParam }};
+            return 307 {{ buildAuthSignURL $externalAuth.SigninURL $externalAuth.SigninURLRedirectParam }};
         }
         {{ end }}
         {{ end }}

diff --git a/test/e2e/annotations/auth.go b/test/e2e/annotations/auth.go
@@ -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")))
 		})
 	})

diff --git a/test/e2e/annotations/satisfy.go b/test/e2e/annotations/satisfy.go
@@ -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")))
 	})
 })