<div dir="ltr"><div dir="ltr">Hello, Amos and Alex,<div><br></div><div>Thank you for sharing your thoughts on this.<br><br>On my part, I see two reasons for implementing support for the <font face="monospace">key=v1,v2,v3</font> format:<br><br>1. As Amos pointed out in the previous email, using this format will reduce the volume of traffic sent from the helper, thereby reducing the processing overhead.<br>However, the impact will be marginal; for 300 groups, the savings would be roughly<font face="monospace"> len("group=") * 300 = 1800 bytes</font>. <br>It would be far more efficient to filter groups during PAC record parsing. This filtering should be performed as early as possible to prevent further processing of unused groups. I intend to spin this suggestion off into a separate discussion thread.</div><div><br>2. Annotation of client connections. Currently, only one tag is supported for client connection annotation: <font face="monospace">clt_conn_tag</font>, and the current handler for this tag does not support multiple occurrences of the same tag in the helper response.<br>However, this issue is elegantly addressed by Amos's PR (<a href="https://github.com/squid-cache/squid/pull/2399/">https://github.com/squid-cache/squid/pull/2399/</a>) and my proposal to use NotePairs filtering (<a href="https://github.com/squid-cache/squid/pull/2399/#issuecomment-4224381496">https://github.com/squid-cache/squid/pull/2399/#issuecomment-4224381496</a>).<br><br>Additionally, custom helpers already have the flexibility to return values in the <font face="monospace">key=v1,v2,v3</font> format, allowing users to retrieve specific values using the<font face="monospace"> -m=, </font>option.<br><br>Furthermore, I’ve reconsidered the use of a comma as a default value separator in helper responses. It seems like a poor choice because we wouldn't be able to distinguish whether the comma is a separator or part of the token itself.<br><br><br>Therefore, I suggest we close this proposal for now and wait for the completion of PR#2399.</div><div><br></div><div>Kind regards,</div><div>     Ankor.</div></div><div><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">вс, 26 апр. 2026 г. в 06:26, Amos Jeffries <<a href="mailto:squid3@treenet.co.nz">squid3@treenet.co.nz</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 24/04/2026 01:42, Alex Rousskov wrote:<br>
> On 2026-04-22 07:24, Andrey K wrote:<br>
>> Hello, Amos and Alex,<br>
>><br>
>> Thank you for the discussion and the proposed formats.<br>
>><br>
>> Since we already support the<br>
>>      key=v1 key=v2<br>
>> format, and the<br>
>>      key=v1,v2<br>
>> format is currently undocumented, I think we should answer two <br>
>> questions first:<br>
>><br>
>> 1. Do we really need to support list-values? It seems Alex thinks we <br>
>> might not need them at all.<br>
> <br>
> If you have to ask, then the answer is "no" or "not yet": We need a very <br>
> compelling use case to add a "list-values" optimization (when we already <br>
> support lists of values). Since you are not sure, and you were the only <br>
> one making/wanting related changes (AFAICT), we evidently lack such a <br>
> case today.<br>
> <br>
> Rule of thumb: Avoid complex optimizations until their expense can be <br>
> justified by a use case (that can also guide their implementation).<br>
> <br>
<br>
Remember that the earlier work by Andrey did bring up a reasonable <br>
rational for doing it anyway.<br>
<br>
Namely that supporting A,B,C lists improves the bandwidth cost of <br>
transmission between helper and Squid.<br>
<br>
Several of the Kerberos use-cases are sending many group names out of <br>
the helper.  So this specific (accept comma-delimited value list) is not <br>
a theoretical optimization.<br>
<br>
<br>
Cheers<br>
Amos<br>
<br>
_______________________________________________<br>
squid-dev mailing list<br>
<a href="mailto:squid-dev@lists.squid-cache.org" target="_blank">squid-dev@lists.squid-cache.org</a><br>
<a href="https://lists.squid-cache.org/listinfo/squid-dev" rel="noreferrer" target="_blank">https://lists.squid-cache.org/listinfo/squid-dev</a><br>
</blockquote></div></div>