Re: [squid-users] 'include' feature

From: Gonzalo Arana <gonzalo.arana@dont-contact.us>
Date: Wed, 9 Jan 2008 11:08:26 -0200

(shifting this thread to squid-dev@)

Adrian,

Some things on the 'include' feature:

* it does not support line wrapping (a line ending with '\\'):
include /dev/null \
    /dev/null
FATAL: Unable to open configuration file: /dev/null \: (2) No such
file or directory

* it takes the rest of the line as a filename:
include /dev/null /dev/null
FATAL: Unable to open configuration file: /dev/null /dev/null: (2) No
such file or directory

* deep recursion triggers a warning, perhaps squid should die in this case?
# in /usr/local/squid2-HEAD/etc/squid.conf
include /dev/null
include /usr/local/squid2-HEAD/etc/squid.conf

2008/01/09 09:39:47| WARNING: can't include /dev/null: includes are
nested too deeply (>16)!
2008/01/09 09:39:47| WARNING: can't include
/usr/local/squid2-HEAD/etc/squid.conf: includes are nested too deeply
(>16)!
and then squid start processing requests.

I've filed bug# 2180 with these issues and my proposed fix.

Once there is a desition on these, I'll forward this to 2.7.

Regards,

On Jan 8, 2008 1:12 PM, Adrian Chadd <adrian@creative.net.au> wrote:
> On Tue, Jan 08, 2008, Gonzalo Arana wrote:
>
> > (I do)**3
> >
> > Will test it tomorow.
>
> Porting the changeset to 2.6 and 2.7 will be trivial..
>
>
>
>
> Adrian
>
>

-- 
Gonzalo A. Arana
Received on Wed Jan 09 2008 - 06:08:29 MST

This archive was generated by hypermail pre-2.1.9 : Fri Feb 01 2008 - 12:00:04 MST