From 2eda21ec6d44fa2f57e72ef0f02ea29c902d871d Mon Sep 17 00:00:00 2001 From: Matthew Holt Date: Fri, 10 Jan 2020 17:02:11 -0700 Subject: http: Remove {...query_string} placeholder, in favor of {...query} I am not sure if the query_string one is necessary or useful yet. We can always add it later if needed. --- modules/caddyhttp/replacer.go | 5 ----- 1 file changed, 5 deletions(-) (limited to 'modules/caddyhttp/replacer.go') diff --git a/modules/caddyhttp/replacer.go b/modules/caddyhttp/replacer.go index 09837be..cea820d 100644 --- a/modules/caddyhttp/replacer.go +++ b/modules/caddyhttp/replacer.go @@ -104,8 +104,6 @@ func addHTTPVarsToReplacer(repl *caddy.Replacer, req *http.Request, w http.Respo return dir, true case "http.request.uri.query": return req.URL.RawQuery, true - case "http.request.uri.query_string": - return "?" + req.URL.RawQuery, true // original request, before any internal changes case "http.request.orig_method": @@ -128,9 +126,6 @@ func addHTTPVarsToReplacer(repl *caddy.Replacer, req *http.Request, w http.Respo case "http.request.orig_uri.query": or, _ := req.Context().Value(OriginalRequestCtxKey).(http.Request) return or.URL.RawQuery, true - case "http.request.orig_uri.query_string": - or, _ := req.Context().Value(OriginalRequestCtxKey).(http.Request) - return "?" + or.URL.RawQuery, true } // hostname labels -- cgit v1.2.3