attr filter "attrsfile" -> "file"

[Arran Cudbard-Bell]

On 8 Jan 2013, at 14:53, Alan DeKok <aland at deployingradius.com> wrote:

> Arran Cudbard-Bell wrote:
>> It's not documented because it's not finalised. Alan D wanted to standardise on using sub section based configuration for the different sections of the server, so there's still some more work to do.
>> 
>> the final config for attrfilter will look something like:
> 
>  Yeah... I'm of two minds about this.  Regularizing the config is good.
> Making it more verbose is annoying.
> 
>> attrfilter {
>> 	authorize {
>> 		file =
>> 	}
>> 	preacct {
>> 		file = 
>> 	}
>> }
> 
>  or even:
> 
> attrfilter {
> 	authorize.file = ...
> 	preacct.file = ...
> 
> }

Sure but that should be done within the conffile API when the configuration files are being parsed.

It'd also allow really compact module definitions like:

attrfilter.authorize.file = <blah>

Which'd be nice.

I don't really want to touch that code though, it's pretty weird.

-Arran