From 9b24a32856bcdfbc16961af9b2e9da63391e9ee8 Mon Sep 17 00:00:00 2001 From: Robert May Date: Mon, 4 Jan 2021 10:18:04 +0000 Subject: Fix for query strings being stripped This ensures the query string is persisted when redirecting to add a trailing slash. --- test/acceptance/serving_test.go | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) (limited to 'test/acceptance/serving_test.go') diff --git a/test/acceptance/serving_test.go b/test/acceptance/serving_test.go index da2843a0..66b5fa47 100644 --- a/test/acceptance/serving_test.go +++ b/test/acceptance/serving_test.go @@ -552,3 +552,22 @@ func doCrossOriginRequest(t *testing.T, spec ListenSpec, method, reqMethod, url rsp.Body.Close() return rsp } + +func TestQueryStringPersistedInSlashRewrite(t *testing.T) { + skipUnlessEnabled(t) + teardown := RunPagesProcess(t, *pagesBinary, listeners, "") + defer teardown() + + rsp, err := GetRedirectPage(t, httpsListener, "group.gitlab-example.com", "project?q=test") + require.NoError(t, err) + defer rsp.Body.Close() + + require.Equal(t, http.StatusFound, rsp.StatusCode) + require.Equal(t, 1, len(rsp.Header["Location"])) + require.Equal(t, "//group.gitlab-example.com/project/?q=test", rsp.Header.Get("Location")) + + rsp, err = GetPageFromListener(t, httpsListener, "group.gitlab-example.com", "project/?q=test") + require.NoError(t, err) + defer rsp.Body.Close() + require.Equal(t, http.StatusOK, rsp.StatusCode) +} -- cgit v1.2.3