Re: rlm_perl and accounting



On Wed 23 Aug 2006 01:49, Alan DeKok wrote:
> > > I see the patch you're referring to, but after rethinking my question,
> > > I think what I'm really trying to do is rewrite $RAD_REQUEST, not
> > > $RAD_REPLY, and it does not appear that I can alter $RAD_REQUEST in any
> > > way - either change or add.
>
>   Hmm... looking into it in a little more detail, I think it would be
> even easier to do it another way.  The code in CVS head has been
> updated to allow for ":=", to over-write existing attributes.  But I
> think it might be even easier to simply use the hashes as-is, and
> replace the existing attribute lists.
>
>   i.e. put the attributes into perl hashes, and then make those perl
> hashes definitive for the new values of the attributes.  This would
> involve throwing away the previous attributes entirely.  So you would
> have to be *very* careful about modifying the hashes, but you would
> have complete flexibility.

That would seem like th logical way to do it, and would certainly make the 
perl code clearer..

-- 

Peter Nixon
http://www.peternixon.net/
PGP Key: http://www.peternixon.net/public.asc

Attachment: pgpGfUenlADkZ.pgp
Description: PGP signature



This archive was generated by a fusion of Pipermail (Mailman edition) and MHonArc.