Re: RegisterESIParser

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Tue, 17 Mar 2009 19:35:27 +1300

Alex Rousskov wrote:
> Hello,
>
> I have moved ESI files to esi/, but it looks like
> RegisterESIParser() leads to the same "unknown to Squid core gets thrown
> out by the linker" problem we have hit with ACLs. Please correct me if I
> am wrong.
>
> There are three parsers in ESI unlike dozens of ACLs, but that does not
> change the nature of the problem. I am going to implement the EsiRegs
> solution, similar to AclRegs. Please let me know if you have better ideas.
>
> N.B. Eventually, we should migrate this to some kind of
> Acl/Esi::InitModule() global function that touches every known ACL/ESI
> parser. This will allow us to move the source of that function into
> acl/esi directories. This would be a slightly cleaner solution, I guess.
>
> Thank you,
>
> Alex.
>

At this point I'm going to say, screw it. Lets at least use the hack
consistently for all these problems and add TODO for a fix in later release.

Amos

-- 
Please be using
   Current Stable Squid 2.7.STABLE6 or 3.0.STABLE13
   Current Beta Squid 3.1.0.6
Received on Tue Mar 17 2009 - 06:34:48 MDT

This archive was generated by hypermail 2.2.0 : Tue Mar 17 2009 - 12:00:04 MDT