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

 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.

>> > 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.

>
>> (if you can think of any other useful scenario than regular
>> splash
>> pages that can be mentioned too.)
>
> Well I've mentioned the possibility of forcing a user to
> re-authenticate, but I'm not sure that I'm correct in saying that. Do
> you think that is possible?

 Mostly the helper is just authorization. I know the distinction between
 authorize and authenticate is a bit narrow here, but this helper in
 particular rides that line with great precision. It will need some
 caveats: "to re-authenticate if %LOGIN parameter and active mode (-a)
 are both used.".

 Applied the patch with that caveat.

 There is quite a bit that could be said about those use-cases. Not sure
 if the wiki or the manual is the best place right now. Probably both in
 the end.

 Amos
Received on Tue Sep 20 2011 - 23:01:12 MDT