Re: [squid-users] change in source code

Amos Jeffries wrote:
>> Henrik Nordstrom wrote:
>>
>>> mån 2008-04-14 klockan 20:13 -0700 skrev Anil Saini:
>>>
>>>
>>>> actually its is thr in squid.conf fine that max limit is 32
>>>> but i increased the limit to 60..and no of dns processes
>>>> increases...but i
>>>> dont know it will effect the squid or not....
>>>>
>>>>
>>> If the number of helpers increased then it worked.
>>>
>>> The limit of 32 was in old versions of Squid, and it seems the comment
>>> in the dns_children description is a leftover.
>>>
>>> Another way is to build Squid without --disable-internal-dns. This will
>>> remove this directive as it isn't even relevant any more..
>>>
>>> Regards
>>> Henrik
>>>
>>>
>> Is this true for 2.6? I had to enable dns_defnames in response to a
>> customer requirement, and found that I had to rebuild 2.6.18 with
>> --disable-internal-dns so dns_defnames would work.
>>
>
> Yes. That option is available for the internal-DNS engine. You just need
> to configure the OS searchpath properly. Squid will append the domain
> suffixes and test the resulting FQDNs in DNS.
>
> In Linux/BSDs the /etc/resolv.conf file has a "search" option taking the
> search paths in the sequence to be tested. First-match is used.
>
> Amos
>
>
Perhaps I'm old fashioned, but my resolv.conf configuration files
generally still use the "domain" specification instead of the "search"
option. I'm successfully using this patch for Squid 3.0, and a very
similar patch on Squid 2.6, to avoid needing to use --disable-internal-dns:

--- src/dns_internal.cc.ORIG 2008-04-16 08:12:07.000000000 -0500
+++ src/dns_internal.cc 2008-04-16 08:13:47.000000000 -0500
@@ -326,6 +326,15 @@
 
                 idnsAddPathComponent(t);
             }
+ } else if (strcasecmp(t, "domain") == 0) {
+ t = strtok(NULL, w_space);
+
+ if (NULL != t) {
+
+ debugs(78, 1, "Adding domain " << t << " from " <<
_PATH_RESCONF);
+
+ idnsAddPathComponent(t);
+ }
         } else if (strcasecmp(t, "options") == 0) {
             while (NULL != t) {
                 t = strtok(NULL, w_space);
Received on Tue Apr 22 2008 - 12:48:57 MDT