Fix BrowserSessionMiddleware RawPath rewrite

Author: cursoragent

lib/browserscope/middleware.go

@@ -33,6 +33,7 @@ func BrowserSessionMiddleware(sessionID, jwt string) option.Middleware {
 					rest = "/" + rest
 				}
 				req.URL.Path = prefix + rest
+				req.URL.RawPath = ""
 			}
 		}
 

lib/browserscope/middleware_test.go

@@ -55,3 +55,27 @@ func TestBrowserSessionMiddlewarePreservesExistingJWT(t *testing.T) {
 func TestBrowserSessionMiddlewareType(t *testing.T) {
 	var _ option.Middleware = BrowserSessionMiddleware("a", "b")
 }
+
+func TestBrowserSessionMiddlewareClearsRawPathOnRewrite(t *testing.T) {
+	mw := BrowserSessionMiddleware("sess1", "")
+	var final *http.Request
+	next := func(req *http.Request) (*http.Response, error) {
+		final = req
+		return nil, nil
+	}
+
+	u, err := url.Parse("https://host/browser/kernel/browsers/sess1/process/%20exec")
+	if err != nil {
+		t.Fatal(err)
+	}
+	req := &http.Request{URL: u}
+
+	_, _ = mw(req, next)
+
+	if final.URL.RawPath != "" {
+		t.Fatalf("raw path should be cleared after rewrite: got %q", final.URL.RawPath)
+	}
+	if got := final.URL.EscapedPath(); got != "/browser/kernel/process/%20exec" {
+		t.Fatalf("escaped path rewrite: got %q", got)
+	}
+}