Search squid archive

Re: External acl on delay_access directive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Alex,


        Thank you for your help. Attached is a patch which seems to fix the issue. Does it seem correct to you ?


Regards,


Luca

On 1/17/19 5:39 PM, Alex Rousskov wrote:
On 1/17/19 9:13 AM, Luca Savarino wrote:

WARNING: ip_list ACL is used in context without an ALE
state. Assuming mismatch.
delay_access 1 allow ip_list
Looks like a Squid bug to me -- Squid should supply ALE (a blob
containing various transaction details) to the delay_access code but
evidently does not.

If you are a developer or can hire a developer to fix this bug, a good
starting point could be the missing ACLFilledChecklist::al
initialization in DelayId::DelayClient().


HTH,

Alex.
_______________________________________________
squid-users mailing list
squid-users@xxxxxxxxxxxxxxxxxxxxx
http://lists.squid-cache.org/listinfo/squid-users
>From 284dbfccf8e5de22721d6480d97e20b43954cec7 Mon Sep 17 00:00:00 2001
From: Luca Savarino <lsavarino@xxxxxxxxx>
Date: Mon, 21 Jan 2019 13:09:26 +0100
Subject: [PATCH] Initializa ALE for delay_pools.

---
 src/DelayId.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/DelayId.cc b/src/DelayId.cc
index fbce786a8..a69d7d91d 100644
--- a/src/DelayId.cc
+++ b/src/DelayId.cc
@@ -97,6 +97,8 @@ DelayId::DelayClient(ClientHttpRequest * http, HttpReply *reply)
 #endif /* FOLLOW_X_FORWARDED_FOR */
             ch.src_addr = r->client_addr;
         ch.my_addr = r->my_addr;
+	ch.al = http->al;
+	ch.al->reply = reply;
 
         if (http->getConn() != NULL)
             ch.conn(http->getConn());
@@ -174,4 +176,3 @@ DelayId::delayRead(DeferredRead const &aRead)
 }
 
 #endif /* USE_DELAY_POOLS */
-
-- 
2.11.0

_______________________________________________
squid-users mailing list
squid-users@xxxxxxxxxxxxxxxxxxxxx
http://lists.squid-cache.org/listinfo/squid-users

[Index of Archives]     [Linux Audio Users]     [Samba]     [Big List of Linux Books]     [Linux USB]     [Yosemite News]

  Powered by Linux