From 8848df9c5d372a559d01512b7a4ef00e38867b55 Mon Sep 17 00:00:00 2001 From: Matthew Holt Date: Mon, 14 Jun 2021 09:03:56 -0600 Subject: Revert "fileserver: Redirect within the original URL (#4179)" This reverts commit f9b54454a19e2b070159ce8d2af76d819658244e. /cc @diamondburned (see #4205) --- modules/caddyhttp/fileserver/browse.go | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'modules/caddyhttp/fileserver/browse.go') diff --git a/modules/caddyhttp/fileserver/browse.go b/modules/caddyhttp/fileserver/browse.go index fa6da68..fc8bddb 100644 --- a/modules/caddyhttp/fileserver/browse.go +++ b/modules/caddyhttp/fileserver/browse.go @@ -47,11 +47,10 @@ func (fsrv *FileServer) serveBrowse(root, dirPath string, w http.ResponseWriter, // URL doesn't end in a trailing slash because hrefs like // "/b/c" on a path like "/a" end up going to "/b/c" instead // of "/a/b/c" - so we have to redirect in this case - oldReq := r.Context().Value(caddyhttp.OriginalRequestCtxKey).(http.Request) - if !strings.HasSuffix(oldReq.URL.Path, "/") { - fsrv.logger.Debug("redirecting to trailing slash to preserve hrefs", zap.String("request_path", oldReq.URL.Path)) - oldReq.URL.Path += "/" - http.Redirect(w, r, oldReq.URL.String(), http.StatusMovedPermanently) + if !strings.HasSuffix(r.URL.Path, "/") { + fsrv.logger.Debug("redirecting to trailing slash to preserve hrefs", zap.String("request_path", r.URL.Path)) + r.URL.Path += "/" + http.Redirect(w, r, r.URL.String(), http.StatusMovedPermanently) return nil } -- cgit v1.2.3