Patch for rlm_attr_filters

Chris Parker cparker at starnetusa.net
Tue Nov 1 22:36:07 CET 2005


On Nov 1, 2005, at 12:15 PM, Alan DeKok wrote:

> Chris Parker <cparker at starnetusa.net> wrote:
>
>> I'd make one change, and that's make it more obvious what the config
>> argument
>> is doing:
>>
>>     defaultaction = drop | allow
>>
>
>   Please see also attr_filter in the CVS head.  As part of my work in
> cleaning up the server internals, I'm removing a function used only by
> attr_filter & policy.  As part of that work, I re-wrote much of
> attr_filter.  It *should* do the same thing, but there's less
> duplicated code.

Yeah, I noted that.  Have looked at it, and it makes sense.  I'll run
it through some tests here, as with all the craziness we do with attrs,
I'm pretty sure any bugs would be poke their heads up pretty quickly.

-Chris
--
Chris Parker
Director, Engineering
StarNet A Service of US LEC

(888)212-0099   Fax (847)963-1302
Wholesale Internet Services     http://www.megapop.net
VoiceEclipse, The Fresh Alternative http://www.voiceeclipse.com

NOTICE: Message is sent IN CONFIDENCE to addressees. It may contain
information that is privileged, proprietary or confidential.




More information about the Freeradius-Devel mailing list