summaryrefslogtreecommitdiff
path: root/modules
diff options
context:
space:
mode:
authorMatthew Holt <mholt@users.noreply.github.com>2020-03-28 19:07:51 -0600
committerMatthew Holt <mholt@users.noreply.github.com>2020-03-28 19:07:51 -0600
commit178ba024fea4db2b91fd159da629f0a8588f119a (patch)
treeb2bb25c2cd68addfefe74df84b7225406000f164 /modules
parente207240f9a6def2e29abf0947a660ab84f858016 (diff)
httpcaddyfile: Put root directive first, before redir and rewrite
See https://caddy.community/t/v2-match-any-path-but-files/7326/8?u=matt If rewrites (or redirects, for that matter) match on file existence, the file matcher would need to know the root of the site. Making this change implies that root directives that depend on rewritten URIs will not work as expected. However, I think this is very uncommon, and am not sure I have ever seen that. Usually, dynamic roots are based on host, not paths or query strings. I suspect that rewrites based on file existence will be more common than roots based on rewritten URIs, so I am moving root to be the first in the list. Users can always override this ordering with the 'order' global option.
Diffstat (limited to 'modules')
-rw-r--r--modules/caddyhttp/caddyhttp.go2
1 files changed, 2 insertions, 0 deletions
diff --git a/modules/caddyhttp/caddyhttp.go b/modules/caddyhttp/caddyhttp.go
index 4fd09a2..6666d3e 100644
--- a/modules/caddyhttp/caddyhttp.go
+++ b/modules/caddyhttp/caddyhttp.go
@@ -199,6 +199,8 @@ func StatusCodeMatches(actual, configured int) bool {
// tlsPlaceholderWrapper is a no-op listener wrapper that marks
// where the TLS listener should be in a chain of listener wrappers.
+// It should only be used if another listener wrapper must be placed
+// in front of the TLS handshake.
type tlsPlaceholderWrapper struct{}
func (tlsPlaceholderWrapper) CaddyModule() caddy.ModuleInfo {