Re: [PATCH] Fix session helper "crashing too rapidly"

On Wed, 2011-09-21 at 11:01 +1200, Amos Jeffries wrote:
> On Tue, 20 Sep 2011 22:26:20 +0100, Andrew Beverley wrote:
> > On Tue, 2011-09-20 at 12:09 +1200, Amos Jeffries wrote:
> >> > Do you mean the -d option to the Squid binary? If so, this doesn't
> >> > seem
> >> > to make any difference; it just prints all the log messages to the
> >> > display as well as the log file.
> >>
> >> -d parameter of the helper binary. stderr is piped to cache.log at
> >> level-0. Thus the need for a separate switch to enable lots of
> >> debug
> >> from the helper.
> >
> > Sorry if I'm being stupid, but do you mean the actual session helper
> > binary or something else? If it's the former, then that obviously
> > just
> > causes an exit due to an invalid option.
> >
>
> Yes I did. Sorry, being unbelievably stupid this week it seems :(.
> Completely overlooked the fact the 'd' parameter and debugs do not
> exist there.

No problem!

However, it still leaves the question: what is the best way to log
errors from the helper. At the moment, even with the patch applied, a
user will still get "The helpers are crashing too rapidly, need help" if
they don't specify a concurrency value. I was trying to fix this, as I
spent hours locating the problem!

>
> >> > Done. Also, the following page should be updated:
> >> >
> >> > http://wiki.squid-cache.org/ConfigExamples/Portal/Splash
> >> >
> >> > I'm happy to do it myself, if you can give me wiki edit rights?
> >>
>
> Enabled.

Thanks, I'll do that shortly.

> Applied the patch with that caveat.
>

Thanks.

And I have another related question, but rather than going high-jacking
this thread, I will start a new one in squid-users :)

Andy
Received on Wed Sep 21 2011 - 22:09:59 MDT