Remove two problematic characters from Routing Transparency test (#1277)

* preserving double quotes but not for HTTP/2
cloudfoundry · Oct 18, 2024 · 084a4f5 · 084a4f5
1 parent aa6a1cb
commit 084a4f5
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/apps/routing_transparency.go b/apps/routing_transparency.go
@@ -44,11 +44,17 @@ var _ = AppsDescribe("Routing Transparency", func() {
 	})
 
 	It("appropriately handles certain reserved/unsafe characters", func() {
-		curlResponse := helpers.CurlApp(Config, appName, "/requesturi/!~^'()$\"?!'()$#!'")
+		curlResponse := helpers.CurlApp(Config, appName, "/requesturi/!~^'()$?!'()$#!'")
 		Expect(curlResponse).To(ContainSubstring("Request"))
 
 		By("preserving all characters")
-		Expect(curlResponse).To(ContainSubstring("/requesturi/!~^'()$\""))
+		Expect(curlResponse).To(ContainSubstring("/requesturi/!~^'()$"))
 		Expect(curlResponse).To(ContainSubstring("Query String is [!'()$]"))
+
+		By("preserving double quotes (not for HTTP/2)")
+		if !Config.GetIncludeHTTP2Routing() {
+			curlResponse = helpers.CurlApp(Config, appName, "/requesturi/\"")
+			Expect(curlResponse).To(ContainSubstring("/requesturi/\""))
+		}
 	})
 })