[RADIATOR] add Attributes when retrying to a new Host in AuthROUNDROBIN

[David Zych]

On 08/19/2014 04:07 PM, Heikki Vatiainen wrote:
> On 08/19/2014 01:00 AM, David Zych wrote:
> 
>> How can I set a new attribute value on a request _each_ time I 
>> attempt to proxy it using AuthRADIUS and friends?  I'm thinking a 
>> "PreForwardHook" would be ideal, but I don't see anything like that 
>> currently implemented.  Is there another solution I'm not seeing, or 
>> if not, would it be possible to add such a hook?
> 
> A hook that is called when a timeout occurs should do the trick.

I agree, a timeout hook would also work nicely for what I need.

>> As an aside: is there any special reason that MaxTargetHosts is 
>> unique to AuthVOLUMEBALANCE?  I would think it would be equally 
>> applicable to all flavors of AuthRADIUS (and in particular I wish it 
>> was implemented in AuthROUNDROBIN).
> 
> In your case you have 4 Hosts, but it would be enough to try e.g., 2?
> With 5 second Timeout even that would take 10 seconds. The use and the
> NAS probably would not wait for much longer.

That's exactly right, and setting MaxTargetHosts 2 would be perfect in
this case.

> I do not know why it has been implemented only for volume balance. Would
> you be interested in trying it for round robin? And maybe the hook too?

I would definitely use both of them.

Thanks!
David