Revert "Normalizes URLs for upstream (#1547)" (#1576)

This reverts commit 5a29572.

Signed-off-by: Sandor Szücs <[email protected]>

proxy/proxy.go

@@ -19,7 +19,6 @@ import (
 	"strings"
 	"time"
 
-	"github.com/dimfeld/httppath"
 	ot "github.com/opentracing/opentracing-go"
 	"github.com/opentracing/opentracing-go/ext"
 	"github.com/zalando/skipper/circuit"
@@ -1463,9 +1462,6 @@ func (p *Proxy) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 		}
 	}()
 
-	// Change /foo/../bar to /bar for matching and passing upstream
-	r.URL.Path = httppath.Clean(r.URL.Path)
-
 	if p.flags.patchPath() {
 		r.URL.Path = rfc.PatchPath(r.URL.Path, r.URL.RawPath)
 	}

routing/matcher.go

@@ -487,8 +487,9 @@ func matchLeaves(leaves leafMatchers, req *http.Request, path, exactPath string)
 // returns the associated value, and the wildcard parameters from the path definition,
 // if any.
 func (m *matcher) match(r *http.Request) (*Route, map[string]string) {
+	// normalize path before matching
 	// in case ignoring trailing slashes, match without the trailing slash
-	path := r.URL.Path
+	path := httppath.Clean(r.URL.Path)
 	exact := path
 	if m.matchingOptions.ignoreTrailingSlash() {
 		path = trimTrailingSlash(path)