mirrors/dendrite
1
0
Fork 0
mirror of https://github.com/matrix-org/dendrite.git synced 2025-12-08 23:43:11 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fix the client API proxy to handle '#' correctly

Browse source
This commit is contained in:
Mark Haines 2017-05-22 19:04:18 +01:00
parent 7f852e3786
commit 9858f4403d
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/github.com/matrix-org/dendrite/cmd/client-api-proxy/main.go
View file

@ -76,7 +76,11 @@ func makeProxy(targetURL string) (*httputil.ReverseProxy, error) {
"url": targetURL, "url": targetURL,
"method": req.Method, "method": req.Method,
}).Print("proxying request") }).Print("proxying request")
newURL, err := url.Parse(targetURL + path) newURL, err := url.Parse(targetURL)
// Set the path separately as we need to preserve '#' characters
// that would otherwise be interpreted as being the start of a URL
// fragment.
newURL.Path += path
if err != nil { if err != nil {
// We already checked that we can parse the URL // We already checked that we can parse the URL
// So this shouldn't ever get hit. // So this shouldn't ever get hit.