From: Willy Tarreau <w@1wt.eu>
Date: Fri, 21 Oct 2016 15:17:18 +0000 (+0200)
Subject: BUG/MINOR: vars: smp_fetch_var() doesn't depend on HTTP but on the session
X-Git-Tag: v1.7-dev5~23
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=87846e42a478fe2aa0fbc1e162ba5fb227be49b7;p=thirdparty%2Fhaproxy.git

BUG/MINOR: vars: smp_fetch_var() doesn't depend on HTTP but on the session

Thus the SMP_USE_HTTP_ANY dependency is incorrect, we have to depend on
SMP_USE_L5_CLI (the session). It's particularly important for session-wide
variables which are kept across HTTP requests. For now there is no impact
but it will make a difference with tcp-request session rules.
---

diff --git a/src/vars.c b/src/vars.c
index 4a0c4ed350..a3dd85ce4b 100644
--- a/src/vars.c
+++ b/src/vars.c
@@ -654,7 +654,7 @@ static int vars_max_size_reqres(char **args, int section_type, struct proxy *cur
 
 static struct sample_fetch_kw_list sample_fetch_keywords = {ILH, {
 
-	{ "var", smp_fetch_var, ARG1(1,STR), smp_check_var, SMP_T_STR, SMP_USE_HTTP_ANY },
+	{ "var", smp_fetch_var, ARG1(1,STR), smp_check_var, SMP_T_STR, SMP_USE_L5CLI },
 	{ /* END */ },
 }};