httpcaddyfile: reject blocks in log_skip directive (#7056)

IwatsukaYura · web-flow · commit 0f209f62eb1f · 2025-06-09T21:56:21.000-06:00
diff --git a/caddyconfig/httpcaddyfile/builtins.go b/caddyconfig/httpcaddyfile/builtins.go
@@ -1166,6 +1166,11 @@ func parseLogSkip(h Helper) (caddyhttp.MiddlewareHandler, error) {
 	if h.NextArg() {
 		return nil, h.ArgErr()
 	}
+
+	if h.NextBlock(0) {
+		return nil, h.Err("log_skip directive does not accept blocks")
+	}
+
 	return caddyhttp.VarsMiddleware{"log_skip": true}, nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1166,6 +1166,11 @@ func parseLogSkip(h Helper) (caddyhttp.MiddlewareHandler, error) {`
`1166`	`1166`	`if h.NextArg() {`
`1167`	`1167`	`return nil, h.ArgErr()`
`1168`	`1168`	`}`
	`1169`	`+`
	`1170`	`+ if h.NextBlock(0) {`
	`1171`	`+ return nil, h.Err("log_skip directive does not accept blocks")`
	`1172`	`+ }`
	`1173`	`+`
`1169`	`1174`	`return caddyhttp.VarsMiddleware{"log_skip": true}, nil`
`1170`	`1175`	`}`
`1171`	`1176`