Ability to set no-redirects=true per path

p53 · Jan 13, 2025 · b55321c · b55321c
1 parent 5082735
commit b55321c
Showing 1 changed file with 39 additions and 39 deletions.
diff --git a/pkg/testsuite/handlers_test.go b/pkg/testsuite/handlers_test.go
@@ -649,45 +649,45 @@ func TestLogoutHandlerGood(t *testing.T) {
 				},
 			},
 		},
-		// {
-		// 	Name:          "TestLogoutWithRedirectQueryParam",
-		// 	ProxySettings: func(_ *config.Config) {},
-		// 	ExecutionSettings: []fakeRequest{
-		// 		{
-		// 			URI:              utils.WithOAuthURI(cfg.BaseURI, cfg.OAuthURI)(constant.LogoutURL) + "?redirect=http://example.com",
-		// 			HasToken:         true,
-		// 			ExpectedCode:     http.StatusSeeOther,
-		// 			ExpectedLocation: "http://example.com",
-		// 		},
-		// 	},
-		// },
-		// {
-		// 	Name: "TestLogoutWithEnabledLogoutRedirect",
-		// 	ProxySettings: func(c *config.Config) {
-		// 		c.EnableLogoutRedirect = true
-		// 	},
-		// 	ExecutionSettings: []fakeRequest{
-		// 		{
-		// 			URI:              utils.WithOAuthURI(cfg.BaseURI, cfg.OAuthURI)(constant.LogoutURL),
-		// 			HasToken:         true,
-		// 			ExpectedCode:     http.StatusSeeOther,
-		// 			ExpectedLocation: "http://127.0.0.1",
-		// 		},
-		// 	},
-		// },
-		// {
-		// 	Name: "TestLogoutWithEmptyRedirectQueryParam",
-		// 	ProxySettings: func(c *config.Config) {
-		// 		c.RedirectionURL = "http://example.com"
-		// 	},
-		// 	ExecutionSettings: []fakeRequest{
-		// 		{
-		// 			URI:          utils.WithOAuthURI(cfg.BaseURI, cfg.OAuthURI)(constant.LogoutURL) + "?redirect=",
-		// 			HasToken:     true,
-		// 			ExpectedCode: http.StatusSeeOther,
-		// 		},
-		// 	},
-		// },
+		{
+			Name:          "TestLogoutWithRedirectQueryParam",
+			ProxySettings: func(_ *config.Config) {},
+			ExecutionSettings: []fakeRequest{
+				{
+					URI:              utils.WithOAuthURI(cfg.BaseURI, cfg.OAuthURI)(constant.LogoutURL) + "?redirect=http://example.com",
+					HasToken:         true,
+					ExpectedCode:     http.StatusSeeOther,
+					ExpectedLocation: "http://example.com",
+				},
+			},
+		},
+		{
+			Name: "TestLogoutWithEnabledLogoutRedirect",
+			ProxySettings: func(c *config.Config) {
+				c.EnableLogoutRedirect = true
+			},
+			ExecutionSettings: []fakeRequest{
+				{
+					URI:              utils.WithOAuthURI(cfg.BaseURI, cfg.OAuthURI)(constant.LogoutURL),
+					HasToken:         true,
+					ExpectedCode:     http.StatusSeeOther,
+					ExpectedLocation: "http://127.0.0.1",
+				},
+			},
+		},
+		{
+			Name: "TestLogoutWithEmptyRedirectQueryParam",
+			ProxySettings: func(c *config.Config) {
+				c.RedirectionURL = "http://example.com"
+			},
+			ExecutionSettings: []fakeRequest{
+				{
+					URI:          utils.WithOAuthURI(cfg.BaseURI, cfg.OAuthURI)(constant.LogoutURL) + "?redirect=",
+					HasToken:     true,
+					ExpectedCode: http.StatusSeeOther,
+				},
+			},
+		},
 	}
 
 	for _, testCase := range testCases {