caddyhttp: Clear out matcher error immediately after grabbing it (#4916)

Co-authored-by: Matthew Holt <mholt@users.noreply.github.com>

1 files changed, 10 insertions, 0 deletions

diff --git a/modules/caddyhttp/vars.go b/modules/caddyhttp/vars.go
index 28d0ddf..e7a7dbb 100644
--- a/modules/caddyhttp/vars.go
+++ b/modules/caddyhttp/vars.go
@@ -301,11 +301,21 @@ func GetVar(ctx context.Context, key string) interface{} {
 // SetVar sets a value in the context's variable table with
 // the given key. It overwrites any previous value with the
 // same key.
+//
+// If the value is nil (note: non-nil interface with nil
+// underlying value does not count) and the key exists in
+// the table, the key+value will be deleted from the table.
 func SetVar(ctx context.Context, key string, value interface{}) {
 	varMap, ok := ctx.Value(VarsCtxKey).(map[string]interface{})
 	if !ok {
 		return
 	}
+	if value == nil {
+		if _, ok := varMap[key]; ok {
+			delete(varMap, key)
+			return
+		}
+	}
 	varMap[key] = value
 }