Re: [squid-users] External acl on delay_access directive

2019-01-21 Thread Alex Rousskov
On 1/21/19 5:22 AM, Luca Savarino wrote: > Attached is a patch which seems to fix the issue. Glad you have a fix that works for you, but this mailing list is not the right place for patch reviews. If you want to submit your changes to the Squid project, I suggest creating a GitHub pull request. T

Re: [squid-users] External acl on delay_access directive

2019-01-21 Thread Luca Savarino
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. Ass

Re: [squid-users] External acl on delay_access directive

2019-01-17 Thread Alex Rousskov
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 eviden

[squid-users] External acl on delay_access directive

2019-01-17 Thread Luca Savarino
Hello,     Having recently upgraded from squid 3.4.8 to squid 4.4, I stumbled into an issue that I tried to simplify the most I could through the attached configuration files and the explanation below.     I would like to use an external acl to set bandwidth limitations for my different use